[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