This is an archive of the discontinued LLVM Phabricator instance.

[gn build] Make HAVE_MALLINFO2 a gn arg, default to false
ClosedPublic

Authored by aeubanks on Jan 26 2022, 9:45 AM.

Details

Summary

D117916 broke some people because some distros are still using a glibc
older than 2.33. Add gn arg llvm_have_mallinfo2 and default to false for
now.

Diff Detail

Event Timeline

aeubanks requested review of this revision.Jan 26 2022, 9:45 AM
aeubanks created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2022, 9:45 AM
peterwaller-arm accepted this revision.Jan 26 2022, 10:14 AM

👍 thanks.

Looks like the build is broken due to PPC_LINUX_DEFAULT_IEEELONGDOUBLE, but reverting 08e535a50797, this differential fixes it.

This revision is now accepted and ready to land.Jan 26 2022, 10:14 AM
This revision was landed with ongoing or failed builds.Jan 26 2022, 10:29 AM
This revision was automatically updated to reflect the committed changes.
pcc added a subscriber: pcc.May 25 2022, 11:48 AM
pcc added inline comments.
llvm/utils/gn/secondary/llvm/include/llvm/Config/BUILD.gn
52

Shouldn't the default be true? If the default assumption that we make in the gn build is that glibc is up to date, users of older glibc should be the ones who need to request the non-default behavior.

Herald added a project: Restricted Project. · View Herald TranscriptMay 25 2022, 11:48 AM
aeubanks added inline comments.May 25 2022, 1:35 PM
llvm/utils/gn/secondary/llvm/include/llvm/Config/BUILD.gn
52

The question is which default inconveniences the fewest people. It probably makes sense to set this to true by default now: https://reviews.llvm.org/D126415