HomePhabricator

[libc++] Simplify the configuration of the C++ ABI library

Authored by ldionne on Apr 1 2020, 11:09 PM.

Description

[libc++] Simplify the configuration of the C++ ABI library

This commit removes support for building against the system libc++abi,
which was supported on Apple platforms. This is basically never what we
want to do, since libc++ and libc++abi are coupled and building a trunk
libc++ against an older libc++abi can lead to incompatibilities (and
good luck debugging them!). It might have made some sense to support
that when the monorepo did not exist, however I don't think this is
anything but a footgun nowadays.

Furthermore, based on the newly-made assumption that we're building
against the monorepo libc++abi, we can simplify the search path logic
for finding libc++abi.

This area of our build system has a lot of technical debt accumulated,
and it's surprisingly difficult to change. We've tried different things
and failed several times in the past. I did test this change on our
Docker image for the build bots and on Apple platforms, however it is
possible that this breaks some unknown configuration, in which case it
should be fine to revert this (so we can try again!).

Details

Committed
ldionneApr 1 2020, 11:21 PM
Parents
rGcbd3969e8cd1: [PPCInstPrinter] Delete an unneeded overload of printBranchOperand. NFC
Branches
Unknown
Tags
Unknown