This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Stop using system ldd to detect libc version
ClosedPublic

Authored by abrachet on May 29 2023, 10:10 PM.

Details

Summary

The system libc may be different from the libc passed in
CMAKE_SYSROOT. Instead of using the ldd in PATH to detect
glibc version, use the features.h header file.

Diff Detail

Event Timeline

abrachet created this revision.May 29 2023, 10:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 29 2023, 10:10 PM
Herald added subscribers: Enna1, dberris. · View Herald Transcript
abrachet requested review of this revision.May 29 2023, 10:10 PM
phosek accepted this revision.Jun 1 2023, 1:18 AM

LGTM

compiler-rt/test/lit.common.cfg.py
586–591

We can skip this entire block altogether when on Android.

607

It might be worth leaving a short inline comment next each flag explaining why it's needed.

This revision is now accepted and ready to land.Jun 1 2023, 1:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 26 2023, 6:41 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript