maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Revert "[RISCV] Enable strict assertions in InsertVSETVLI data flow" 

This reverts commit 79a66ec97b.

The stronger asserts served their purpose; I stumbled across another bug.  Will reapply once this one is also fixed.

The bug appears to be a variant of a previous one:
* We mutate an instruction in one block.
* That mutation changes the phase3 results of another block.

This is very similiar to a previous issue, except cross block instead of within a single block.
This commit is contained in:
Philip Reames 2022-05-17 15:51:41 -07:00
parent 996834e681
commit d4545e6fa0
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
View File

@ -38,7 +38,7 @@ static cl::opt<bool> DisableInsertVSETVLPHIOpt(
cl::desc("Disable looking through phis when inserting vsetvlis."));
static cl::opt<bool> UseStrictAsserts(
"riscv-insert-vsetvl-strict-asserts", cl::init(true), cl::Hidden,
"riscv-insert-vsetvl-strict-asserts", cl::init(false), cl::Hidden,
cl::desc("Enable strict assertion checking for the dataflow algorithm"));
namespace {