[llvm] r291516 - Add an assert for hasLoopInvariantOperands
Xin Tong via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 9 16:39:49 PST 2017
Author: trentxintong
Date: Mon Jan 9 18:39:49 2017
New Revision: 291516
URL: http://llvm.org/viewvc/llvm-project?rev=291516&view=rev
Log:
Add an assert for hasLoopInvariantOperands
Summary: Add an assert for hasLoopInvariantOperands
Reviewers: danielcdh, sanjoy
Subscribers: mzolotukhin, llvm-commits
Differential Revision: https://reviews.llvm.org/D28501
Modified:
llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp?rev=291516&r1=291515&r2=291516&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LoopSink.cpp Mon Jan 9 18:39:49 2017
@@ -283,6 +283,9 @@ static bool sinkLoopInvariantInstruction
// sinked.
for (auto II = Preheader->rbegin(), E = Preheader->rend(); II != E;) {
Instruction *I = &*II++;
+ // No need to check for instruction's operands are loop invariant.
+ assert(L.hasLoopInvariantOperands(I) &&
+ "Insts in a loop's preheader should have loop invariant operands!");
if (!canSinkOrHoistInst(*I, &AA, &DT, &L, &CurAST, nullptr))
continue;
if (sinkInstruction(L, *I, ColdLoopBBs, LoopBlockNumber, LI, DT, BFI))
More information about the llvm-commits
mailing list