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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
👍 thanks.
Looks like the build is broken due to PPC_LINUX_DEFAULT_IEEELONGDOUBLE, but reverting 08e535a50797, this differential fixes it.
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. |
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 |
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.