This is an archive of the discontinued LLVM Phabricator instance.

[libunwind] Use the correct variable name for target triple in lit
AbandonedPublic

Authored by phosek on Dec 4 2017, 4:30 PM.

Details

Summary

The variable representing target triple in libunwind is named
LIBUNWIND_TARGET_TRIPLE, not TARGET_TRIPLE.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Dec 4 2017, 4:30 PM

There's a: set(TARGET_TRIPLE ...) in each of these runtimes' CMakeLists.txt...

(i.e. you should delete that, since this is the last dependency on it AFAICT)

phosek added a comment.Dec 4 2017, 4:52 PM

Actually it seems to be missing in libunwind, but I see it in libc++abi and libc++ (I missed that one). Do you want me to remove it from those two or add it to libunwind? It's a little strange to be using both LIBUNWIND_TARGET_TRIPLE and TARGET_TRIPLE in the same file.

I think just remove it from them.

compnerd accepted this revision.Dec 5 2017, 11:27 AM

I'm okay with this change in principle, but Im worried that this may break the buildbots. Please ensure that they remain green after this change.

This revision is now accepted and ready to land.Dec 5 2017, 11:27 AM

After some more digging through libc++ test suite, I think I understand now why these are different variables. It may be possible to remove these and just rely on plain CMake variables but this is likely going to require some more CMake and lit surgery, so for now I'm going to land D40820 which is an alternative solution and revisit this later after I cleanup libc++ test suite.

phosek abandoned this revision.Feb 4 2019, 12:27 PM

This is no longer needed after D57670 landed.

Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2019, 12:27 PM