[PATCH] D130252: [RISCV] Precommit test for D130251

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 21 06:56:03 PDT 2023


RKSimon added inline comments.
Herald added subscribers: wangpc, jobnoorman, luke.


================
Comment at: llvm/test/CodeGen/RISCV/rotl-rotr.ll:402
+; RV64ZBB-NEXT:    ret
+  %a = or i32 %y, 64
+  %b = shl i32 %x, %a
----------------
@craig.topper @pcwang-thead You might want to review theses tests as the out of bounds shifts means they generate poison (which is exploited by rGae60706da07a128) - maybe use fshl/fshr intrinsics?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130252



More information about the llvm-commits mailing list