This is an archive of the discontinued LLVM Phabricator instance.

Remove duplicate LLVM_NATIVE_ from llvm's config.h
ClosedPublic

Authored by dim on Jun 4 2017, 12:21 PM.

Details

Summary

Since LLVM_NATIVE_ARCH, LLVM_NATIVE_ASMPARSER, LLVM_NATIVE_ASMPRINTER,
LLVM_NATIVE_DISASSEMBLER, LLVM_NATIVE_TARGET, LLVM_NATIVE_TARGETINFO and
LLVM_NATIVE_TARGETMC are already defined in llvm-config.h, there seems
to be no reason to also define them in config.h. Also, I can only find
usage of these macros in files that include llvm-config.h.

So let's remove the duplicated macros from config.h.

Diff Detail

Repository
rL LLVM

Event Timeline

dim created this revision.Jun 4 2017, 12:21 PM
chapuni added a subscriber: chapuni.Jun 4 2017, 4:02 PM
rnk accepted this revision.Jun 5 2017, 5:53 AM

lgtm, ship it :)

This revision is now accepted and ready to land.Jun 5 2017, 5:53 AM
This revision was automatically updated to reflect the committed changes.