This is an archive of the discontinued LLVM Phabricator instance.

[bolt][llvm][cmake] Use `CMAKE_INSTALL_LIBDIR` too
ClosedPublic

Authored by Ericson2314 on Aug 20 2022, 7:48 AM.

Details

Summary

Working back towards D130586.

Bolt didn't use LLVM_LIBDIR_SUFFIX before, and has no in-tree reverse
dependencies, it seems easier to add.

The change in LLVM itself is to prevent some unexpected lib64 from
cropping up due to the CMAKE_INSTALL_LIBDIR defaulting logic.

Diff Detail

Event Timeline

Ericson2314 created this revision.Aug 20 2022, 7:48 AM
Herald added a reviewer: Amir. · View Herald Transcript
Herald added a reviewer: maksfb. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Ericson2314 requested review of this revision.Aug 20 2022, 7:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 20 2022, 7:48 AM

This is a subset of D130586, which was approved, and unlike the original it appears to be passing CI all the way. I'll just go ahead and merge then so I can continue "bisecting".

This revision was not accepted when it landed; it landed in state Needs Review.Aug 20 2022, 10:08 AM
This revision was automatically updated to reflect the committed changes.