diff options
| author | Bjorn Pettersson <bjorn.a.pettersson@ericsson.com> | 2019-08-09 07:08:25 +0000 |
|---|---|---|
| committer | Bjorn Pettersson <bjorn.a.pettersson@ericsson.com> | 2019-08-09 07:08:25 +0000 |
| commit | d218a3326e7152469f0c6837f682645ffff0fcbc (patch) | |
| tree | 93038296a08233730971bb97d592052fceac6de5 /llvm/lib | |
| parent | 6179175551c04877cc580e2973b239e9d2e7d394 (diff) | |
| download | bcm5719-llvm-d218a3326e7152469f0c6837f682645ffff0fcbc.tar.gz bcm5719-llvm-d218a3326e7152469f0c6837f682645ffff0fcbc.zip | |
[InstSimplify] Report "Changed" also when only deleting dead instructions
Summary:
Make sure that we report that changes has been made
by InstSimplify also in situations when only trivially
dead instructions has been removed. If for example a call
is removed the call graph must be updated.
Bug seem to have been introduced by llvm-svn r367173
(commit 02b9e45a7e4b81), since the code in question
was rewritten in that commit.
Reviewers: spatel, chandlerc, foad
Reviewed By: spatel
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65973
llvm-svn: 368401
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/InstSimplifyPass.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/InstSimplifyPass.cpp b/llvm/lib/Transforms/Scalar/InstSimplifyPass.cpp index c31d0e74641..920b12b8a6a 100644 --- a/llvm/lib/Transforms/Scalar/InstSimplifyPass.cpp +++ b/llvm/lib/Transforms/Scalar/InstSimplifyPass.cpp @@ -45,6 +45,7 @@ static bool runImpl(Function &F, const SimplifyQuery &SQ, // Don't waste time simplifying dead/unused instructions. if (isInstructionTriviallyDead(&I)) { DeadInstsInBB.push_back(&I); + Changed = true; } else if (!I.use_empty()) { if (Value *V = SimplifyInstruction(&I, SQ, ORE)) { // Mark all uses for resimplification next time round the loop. |
