This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] Added check for NUW flag.
Needs ReviewPublic

Authored by IlyaKrotov on May 8 2015, 10:25 AM.

Details

Reviewers
dblaikie
djasper
Summary

Check if we can add NUW flag. If so, set this.

Diff Detail

Event Timeline

IlyaKrotov updated this revision to Diff 25345.May 8 2015, 10:25 AM
IlyaKrotov retitled this revision from to [InstCombine] Added check for NUW flag..
IlyaKrotov updated this object.
IlyaKrotov edited the test plan for this revision. (Show Details)
IlyaKrotov added reviewers: dblaikie, djasper.
IlyaKrotov added a subscriber: Unknown Object (MLST).
sanjoy added a subscriber: sanjoy.May 8 2015, 4:23 PM

As I was looking through this, I realized that the nsw case is incorrect: http://reviews.llvm.org/D9629