[PATCH] D93509: [IndVars] Fix adding trunc instructions for unwind blocks

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 20:12:17 PST 2020


yrouban created this revision.
yrouban added reviewers: mkazantsev, skatkov.
Herald added a subscriber: hiraditya.
yrouban requested review of this revision.
Herald added a project: LLVM.

We should not insert truncation before landing pads.
Fix the insertion point.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93509

Files:
  llvm/lib/Transforms/Utils/SimplifyIndVar.cpp


Index: llvm/lib/Transforms/Utils/SimplifyIndVar.cpp
===================================================================
--- llvm/lib/Transforms/Utils/SimplifyIndVar.cpp
+++ llvm/lib/Transforms/Utils/SimplifyIndVar.cpp
@@ -1654,7 +1654,7 @@
     assert(LoopExitingBlock && L->contains(LoopExitingBlock) &&
            "Not a LCSSA Phi?");
     WidePN->addIncoming(WideBO, LoopExitingBlock);
-    Builder.SetInsertPoint(User->getParent()->getFirstNonPHI());
+    Builder.SetInsertPoint(&*User->getParent()->getFirstInsertionPt());
     auto *TruncPN = Builder.CreateTrunc(WidePN, User->getType());
     User->replaceAllUsesWith(TruncPN);
     DeadInsts.emplace_back(User);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93509.312678.patch
Type: text/x-patch
Size: 671 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201218/31217349/attachment.bin>


More information about the llvm-commits mailing list