[PATCH] D154915: [ARM][AArch64] Add ARM specific builtin for clz that is not undefined for 0 in ubsan.

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 10 20:24:45 PDT 2023


craig.topper added inline comments.


================
Comment at: clang/test/CodeGen/arm_acle.c:351
 //
 long test_clzl(long t) {
   return __clzl(t);
----------------
__builtin_clzl returned a signed int, but the new builtin returns an unsigned int so the cast changed.


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