This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by phosek on Apr 13 2017, 7:57 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. The compile flag handling had a typo which caused these flag not to be properly include. 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:57 PM
compnerd accepted this revision.Apr 14 2017, 10:00 AM
compnerd added inline comments.
CMakeLists.txt
190

I think that the space is easily worked with. We just need to append two items instead of one so that the quoting is handled properly. I do really prefer the single dash version for the -target flag.

This revision is now accepted and ready to land.Apr 14 2017, 10:00 AM
This revision was automatically updated to reflect the committed changes.