This is an archive of the discontinued LLVM Phabricator instance.

[libc++abi][ci] Add a from-scratch config for libc++abi on Apple/system
ClosedPublic

Authored by ldionne on Oct 6 2021, 3:37 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Restricted Project
Restricted Project
Commits
rGd1e0f02e0ba9: [libc++abi][ci] Add a from-scratch config for libc++abi on Apple/system
Summary

I came across an issue where since we build the library for Apple with
the install name directory being /usr/lib, which means that if we don't
run the tests with DYLD_LIBRARY_PATH, we'll end up loading the
system-provided libc++abi when running the tests. That wreaks havoc.

Instead of fixing it in the legacy config file, this commit introduces
an Apple libc++abi config file that does the right thing.

Diff Detail

Event Timeline

ldionne created this revision.Oct 6 2021, 3:37 PM
ldionne requested review of this revision.Oct 6 2021, 3:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 6 2021, 3:37 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne updated this revision to Diff 378770.Oct 11 2021, 1:08 PM

Rebase onto main

Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptOct 11 2021, 1:08 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a subscriber: mgorny. · View Herald Transcript
ldionne updated this revision to Diff 379195.Oct 12 2021, 3:02 PM
ldionne retitled this revision from [runtimes][ci] Run libc++abi and libunwind on from-scratch configs to [libc++abi][ci] Add a from-scratch config for libc++abi on Apple/system.
ldionne edited the summary of this revision. (Show Details)

Repurpose patch

ldionne updated this revision to Diff 379206.Oct 12 2021, 3:30 PM

Fix test failures.

ldionne accepted this revision.Oct 13 2021, 5:03 AM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 13 2021, 5:08 AM
This revision was automatically updated to reflect the committed changes.