diff options
| author | Johannes Doerfert <johannes@jdoerfert.de> | 2021-07-19 15:31:10 -0500 |
|---|---|---|
| committer | Johannes Doerfert <johannes@jdoerfert.de> | 2021-07-26 23:33:36 -0500 |
| commit | 25a3130d89f00f53a23f7fc38250d5dc43e29cfb (patch) | |
| tree | a1e197a774693c1f994d93ff890a45a5e987e598 /llvm/lib/Transforms/Utils/LoopUtils.cpp | |
| parent | 41bd26dff9869a0d72f95ed750a82136f1bccc81 (diff) | |
| download | llvm-25a3130d89f00f53a23f7fc38250d5dc43e29cfb.zip llvm-25a3130d89f00f53a23f7fc38250d5dc43e29cfb.tar.gz llvm-25a3130d89f00f53a23f7fc38250d5dc43e29cfb.tar.bz2 | |
[Local] Do not introduce a new `llvm.trap` before `unreachable`
This is the second attempt to remove the `llvm.trap` insertion after
https://reviews.llvm.org/rGe14e7bc4b889dfaffb7180d176a03311df2d4ae6
reverted the first one. It is not clear what the exact issue was back
then and it might already be gone by now, it has been >5 years after
all.
Replaces D106299.
Differential Revision: https://reviews.llvm.org/D106308
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUtils.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/LoopUtils.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/LoopUtils.cpp b/llvm/lib/Transforms/Utils/LoopUtils.cpp index 15a87dd..e4d78f9 100644 --- a/llvm/lib/Transforms/Utils/LoopUtils.cpp +++ b/llvm/lib/Transforms/Utils/LoopUtils.cpp @@ -723,8 +723,8 @@ void llvm::breakLoopBackedge(Loop *L, DominatorTree &DT, ScalarEvolution &SE, auto *BackedgeBB = SplitEdge(Latch, Header, &DT, &LI, MSSAU.get()); DomTreeUpdater DTU(&DT, DomTreeUpdater::UpdateStrategy::Eager); - (void)changeToUnreachable(BackedgeBB->getTerminator(), /*UseTrap*/false, - /*PreserveLCSSA*/true, &DTU, MSSAU.get()); + (void)changeToUnreachable(BackedgeBB->getTerminator(), + /*PreserveLCSSA*/ true, &DTU, MSSAU.get()); // Erase (and destroy) this loop instance. Handles relinking sub-loops // and blocks within the loop as needed. |