[compiler-rt] [llvm] scudo: default enabled dealloc_type_mismatch on new platforms (PR #146739)
Justin King via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 8 08:17:12 PDT 2025
jcking wrote:
> What is the motivation for doing this?
>
> There are more than just android, trusty and fuchsia and having this strange check to set the flag feels like the wrong approach. Is there a reason this can't be a config change with the default off. Then you could add the config parameter as enabled in allocator_config.h to all of the configs.
>
> This feels like something that's either on all of the time, or off all of the time, not enabled at run time.
The goal is to just have `dealloc_type_mismatch` be `true` by default. However AFAIK it is off by default due to some bad software on Android. Just because it is off by default for Android, doesn't mean future platforms should make the same decision. Are you saying to move to a preprocessor macro that is defined by CMake, and then have downstream users change it?
https://github.com/llvm/llvm-project/pull/146739
More information about the llvm-commits
mailing list