This is an archive of the discontinued LLVM Phabricator instance.

Remove `LIBC_INSTALL_PREFIX`
ClosedPublic

Authored by Ericson2314 on Jul 9 2021, 2:46 PM.

Details

Summary

This matches the decision made in D99697.

It also shouldn't reintroduce the issue fixed in D99636.

The variable was originally introduced in
b22f448c21e718a3b6219df89169f38d436189c6 but is not essential to that
change.

Once we finish adding GnuInstallDirs support in D100810 and D99484,
setting CMAKE_INSTALL_LIBDIR would also work to change the
installation directory (though for more than libc).

GnuInstallDirs support also brings up an issue which is avoided if
variables like LIBC_INSTALL_PREFIX don't exist. Because the
GnuInstallDirs variables can be absolute paths, it is a bit unclear
how the per-project prefixes would work: does the project-agnostic
role-specific variable (e.g. CMAKE_INSTALL_LIBDIR), or project-specfic
role-agnostic (e.g. LIBC_INSTALL_PREFIX) take priority? Each is more
specific than the other on one axis, but not the other.

Diff Detail

Event Timeline

Ericson2314 created this revision.Jul 9 2021, 2:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2021, 2:46 PM
Ericson2314 requested review of this revision.Jul 9 2021, 2:46 PM

@phosek also mind reviewing this one? An alternative would be the fancy concatenation logic I just did for compiler-rt, but I think that's a bit overkill when there is currently just one variable to be prepended.

phosek accepted this revision.Jul 20 2021, 9:48 PM

LGTM

This revision is now accepted and ready to land.Jul 20 2021, 9:48 PM
This revision was automatically updated to reflect the committed changes.