[PATCH] D154915: [ARM][AArch64] Add ARM specific builtin for clz that is not undefined for 0 in ubsan.
Tomas Matheson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 11 07:38:53 PDT 2023
tmatheson added a comment.
I would prefer the simplicity of adding a check in the intrinsic itself, rather than adding the target-specific builtins. Slightly worse codegen at -O0 doesn't matter imho. However I don't feel very strongly about it, so if others are happy with this then LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154915/new/
https://reviews.llvm.org/D154915
More information about the cfe-commits
mailing list