[PATCH] D101472: [RISCV] Teach computeKnownBits that vsetvli returns number less than 2^31.

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 01:04:52 PDT 2021


frasercrmck accepted this revision.
frasercrmck added a comment.

LGTM otherwise, cheers!



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:5789
+    unsigned IntNo = Op.getConstantOperandVal(1);
+    switch (IntNo) {
+    case Intrinsic::riscv_vsetvli:
----------------
Should we have an explicit `default`? I know some compilers warn about that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101472



More information about the llvm-commits mailing list