This is an archive of the discontinued LLVM Phabricator instance.

Using ARMTargetParser in Clang
ClosedPublic

Authored by rengolin on May 7 2015, 3:19 AM.

Details

Summary

This is a starting point for using the TargetParser in Clang, in a simple
enough part of the code that can be used without disrupting the crazy
platform support that we need to be compatible with other toolchains.

Also adding a few FIXME on obvious places that need replacing, but those
cases will indeed break a few of the platform assumptions, as arch/cpu names
change multiple times in the driver.

Finally, I'm changing the "neon-vfpv3" behaviour to match standard NEON, since
-mfpu=neon implies vfpv3 by default in both Clang and LLVM. That option
string is still supported as an alias to "neon".

This patch depends on D9435 for support in LLVM.

Diff Detail

Event Timeline

rengolin updated this revision to Diff 25148.May 7 2015, 3:19 AM
rengolin retitled this revision from to Using ARMTargetParser in Clang.
rengolin updated this object.
rengolin edited the test plan for this revision. (Show Details)
rengolin set the repository for this revision to rL LLVM.
rengolin added subscribers: Unknown Object (MLST), Unknown Object (MLST).
rengolin updated this revision to Diff 25154.May 7 2015, 4:01 AM
rengolin removed rL LLVM as the repository for this revision.

Removing unnecessary "clang" match in tests, since it's only one command line per execution.

rengolin updated this revision to Diff 25307.May 8 2015, 4:32 AM

Rename ARMTargetParser.h/cpp to TargetParser.h/cpp

rnk accepted this revision.May 8 2015, 11:25 AM
rnk edited edge metadata.

lgtm

This revision is now accepted and ready to land.May 8 2015, 11:25 AM
rengolin closed this revision.May 8 2015, 2:09 PM

Thanks! r236901.