[PATCH] D67661: [RISCV] Headers: Add Bitmanip extension Clang header files and rvintrin.h

Lewis Revill via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 3 08:23:21 PST 2019


lewis-revill added inline comments.


================
Comment at: clang/lib/Headers/rv32bintrin-builtins.h:27
+_rv32_clz(const uint_xlen_t rs1) {
+  // Calling these builtins with 0 results in undefined behaviour.
+  if (rs1 == 0) {
----------------
Does GCC perform this check before calling the builtin?


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

https://reviews.llvm.org/D67661





More information about the cfe-commits mailing list