[PATCH] D146324: scudo: Don't define mallinfo2 on Android.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 14:29:19 PDT 2023


pcc added a comment.

In D146324#4203219 <https://reviews.llvm.org/D146324#4203219>, @cferris wrote:

> Is it worth examining how we do this on Android? For example, should we remove the mallinfo2 alias for mallinfo, so that we don't have to make this change?

If you removed the mallinfo2 alias and added a definition of mallinfo2 to Bionic, then it would mean that newer NDK versions will increase the API level requirement for mallinfo2 from baseline to whichever version of Android you added it in. So from an NDK user perspective, that doesn't seem like a good idea.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146324/new/

https://reviews.llvm.org/D146324



More information about the llvm-commits mailing list