[PATCH] D97970: [RISCV] Make the hasStdExtM() check in RISCVInstrInfo::getVLENFactoredAmount emit a diagnostic rather than an assert.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 8 09:10:20 PST 2021
craig.topper added a comment.
In D97970#2610621 <https://reviews.llvm.org/D97970#2610621>, @frasercrmck wrote:
> This is better than an assert, but do we absolutely require M? Couldn't we fall back to compiler-rt, for instance?
Maybe. I'm not sure if I we're able to inject a call at the point in the compiler where this is used.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97970/new/
https://reviews.llvm.org/D97970
More information about the llvm-commits
mailing list