[PATCH] D150262: Disable sanitizer's on ifunc resolvers.

Farzon Lotfi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 15 16:31:35 PDT 2023


farzon added a comment.

I feel like this change will break our use case and do so in a nontransparent way. We want to be able to use sanitizers even if IFUNC indirection is defined.

So if the compiler toolchain is now going to disable santizer instrumentation overriding the preferences of the person building the software then that is going to regress https://bugs.chromium.org/p/chromium/issues/detail?id=1454613 again and make it so we can't fuzz on arm64.

Instead can we not just fail the build? That way the CI will be the enforcer and the next developer that comes along will have to create a happy path to support sanitizer builds.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150262



More information about the cfe-commits mailing list