This is an archive of the discontinued LLVM Phabricator instance.

[CMake][libcxx] Fix the --target and --gcc-toolchain flag handling
ClosedPublic

Authored by phosek on Apr 13 2017, 7:54 PM.

Details

Summary

CMake has the problem with the single dash variant because of the space, so use the double dash with equal sign version. We also don't have to pass the target triple when checking for compiler-rt since that flag is already included in compile flags now.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Apr 13 2017, 7:54 PM

Ping, can someone please take a look?

EricWF accepted this revision.Apr 15 2017, 2:53 PM
This revision is now accepted and ready to land.Apr 15 2017, 2:53 PM

Thanks, can you also please take a look at D32069 which is the same change for libc++abi?

This revision was automatically updated to reflect the committed changes.