This is an archive of the discontinued LLVM Phabricator instance.

[cmake] Move HandleOutOfTreeLLVM to common cmake utils
ClosedPublic

Authored by Ericson2314 on Jan 18 2022, 3:03 PM.

Details

Reviewers
phosek
compnerd
beanz
Ericson2314
Group Reviewers
Restricted Project
Restricted Project
Restricted Project
Commits
rG429a717ea5ec: [cmake] Move HandleOutOfTreeLLVM to common cmake utils
Summary

This is better than libunwind and libcxxabi fishing it out of libcxx's
module directory.

It is done in prepartion for a better version of D117537 which deduplicates
CMake logic instead of just renaming to avoid a name clash.

Diff Detail

Event Timeline

Ericson2314 created this revision.Jan 18 2022, 3:03 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2022, 3:03 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Ericson2314 requested review of this revision.Jan 18 2022, 3:03 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 18 2022, 3:03 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Ericson2314 retitled this revision from Move HandleOutOfTreeLLVM to common cmake utils to [cmake] Move HandleOutOfTreeLLVM to common cmake utils.Jan 18 2022, 3:39 PM
phosek accepted this revision.Jan 19 2022, 1:19 PM

LGTM we should see if we can also use HandleOutOfTreeLLVM.cmake in compiler-rt in a follow-up change.

Ericson2314 accepted this revision.Jan 19 2022, 2:03 PM

Hmm not sure why it's not going to the accepted state. I guess I did something odd when I marked it needs more review before.

LGTM we should see if we can also use HandleOutOfTreeLLVM.cmake in compiler-rt in a follow-up change.

That is indeed what I ended up doing! :)

This revision was not accepted when it landed; it landed in state Needs Review.Jan 19 2022, 2:05 PM
This revision was automatically updated to reflect the committed changes.