This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Move the setting of LLVM_COMPILER_IS_GCC_COMPATIBLE to a separate file
ClosedPublic

Authored by john.brawn on Sep 28 2015, 9:04 AM.

Details

Summary

Currently LLVM_COMPILER_IS_GCC_COMPATIBLE is set as a side-effect of determining the stdlib to use in HandleLLVMStdlib, which causes problems when attempting to use AddLLVM from an installed LLVM toolchain, as HandleLLVMStdlib is not used. Move the setting of this variable into DetermineGCCCompatible and include that from both AddLLVM and HandleLLVMStdlib.

Diff Detail

Repository
rL LLVM

Event Timeline

john.brawn updated this revision to Diff 35878.Sep 28 2015, 9:04 AM
john.brawn retitled this revision from to [CMake] Move the setting of LLVM_COMPILER_IS_GCC_COMPATIBLE to a separate file.
john.brawn updated this object.
john.brawn set the repository for this revision to rL LLVM.
john.brawn added a subscriber: llvm-commits.
chapuni edited edge metadata.Sep 28 2015, 4:52 PM

Looks good.

That said, I think rather we may reorganize handling individual options.
I don't have strong opinions for this.

This revision was automatically updated to reflect the committed changes.