This is an archive of the discontinued LLVM Phabricator instance.

Add an assert for hasLoopInvariantOperands
ClosedPublic

Authored by trentxintong on Jan 9 2017, 4:41 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

trentxintong retitled this revision from to Add an assert for hasLoopInvariantOperands.
trentxintong updated this object.
trentxintong added reviewers: danielcdh, sanjoy.
trentxintong added a subscriber: llvm-commits.
sanjoy accepted this revision.Jan 9 2017, 4:46 PM
sanjoy edited edge metadata.
sanjoy added inline comments.
lib/Transforms/Scalar/LoopSink.cpp
288 ↗(On Diff #83741)

I'd say something about the preheader in the assertion message: "Instructions in a loop's preheader should have loop invariant operands!" or something like that.

This revision is now accepted and ready to land.Jan 9 2017, 4:46 PM
trentxintong edited edge metadata.

Address comments.

This revision was automatically updated to reflect the committed changes.