[PATCH] D152250: [Clang][RISCV] Add test cases for intrinsics clz/ctz codegen when has extension zbb/xtheadbb

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/test/CodeGen/RISCV/rvb-intrinsics/riscv64-xtheadbb.c:52
+//
+long clz_64_generic(long a) {
+  return __builtin_clzl(a);
----------------
Should the return type be int to match __builtin_clzl?


================
Comment at: clang/test/CodeGen/RISCV/rvb-intrinsics/riscv64-zbb.c:51
+//
+long clz_64_generic(long a) {
+  return __builtin_clzl(a);
----------------
Should the return type be int to match __builtin_clzl


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152250

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