Skip to content

Commit 1f06398

Browse files
committed
Reapply "[RISCV] Enable strict assertions in InsertVSETVLI data flow"
be2cb8 fixes the case which triggered the revert. Reapply, and let's see if anything else falls out. Original commit message: These asserts are believed to hold after several recent miscompiles have been fixed. If you see an assertion failure on this change, please toggle the default back and make sure you file a bug with a reproducer. We may have as yet uncaught miscompiles lurking in this code. Differential Revision: https://reviews.llvm.org/D125271
1 parent 4baae16 commit 1f06398

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ static cl::opt<bool> DisableInsertVSETVLPHIOpt(
3838
cl::desc("Disable looking through phis when inserting vsetvlis."));
3939

4040
static cl::opt<bool> UseStrictAsserts(
41-
"riscv-insert-vsetvl-strict-asserts", cl::init(false), cl::Hidden,
41+
"riscv-insert-vsetvl-strict-asserts", cl::init(true), cl::Hidden,
4242
cl::desc("Enable strict assertion checking for the dataflow algorithm"));
4343

4444
namespace {

0 commit comments

Comments
 (0)