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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to