diff options
| author | Craig Topper <craig.topper@intel.com> | 2019-08-19 18:02:24 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2019-08-19 18:02:24 +0000 |
| commit | 93c27871936c797380aef9d47a969c088edb0bff (patch) | |
| tree | bf580d797afb26537669045c7a16c7471e15e517 /llvm/lib/CodeGen/CodeGenPrepare.cpp | |
| parent | ab00f237ac9db0fb91120bf4a5454f44ad31cf7a (diff) | |
| download | bcm5719-llvm-93c27871936c797380aef9d47a969c088edb0bff.tar.gz bcm5719-llvm-93c27871936c797380aef9d47a969c088edb0bff.zip | |
[CGP] Remove ModifiedDT from the makeBitReverse loop
I don't think anything in this loop modifies the control flow and we don't restart any iteration after setting the flag.
This code was added in http://reviews.llvm.org/D16893 but looking at the test case added there the code that caused the dominator tree to change was merging blocks with their predecessor not the bitreverse optimization.
Differential Revision: https://reviews.llvm.org/D66366
llvm-svn: 369283
Diffstat (limited to 'llvm/lib/CodeGen/CodeGenPrepare.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/CodeGenPrepare.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/CodeGenPrepare.cpp b/llvm/lib/CodeGen/CodeGenPrepare.cpp index 7da810c40c4..c6edbb90e00 100644 --- a/llvm/lib/CodeGen/CodeGenPrepare.cpp +++ b/llvm/lib/CodeGen/CodeGenPrepare.cpp @@ -7104,7 +7104,6 @@ bool CodeGenPrepare::optimizeBlock(BasicBlock &BB, bool &ModifiedDT) { for (auto &I : reverse(BB)) { if (makeBitReverse(I, *DL, *TLI)) { MadeBitReverse = MadeChange = true; - ModifiedDT = true; break; } } |
