This is an archive of the discontinued LLVM Phabricator instance.

[CVP] Set NSW/NUW flags when simplifying with.overflow
ClosedPublic

Authored by nikic on Apr 11 2019, 2:18 PM.

Details

Summary

When CVP determines that a with.overflow intrinsic cannot overflow, it currently inserts a simple add/sub. As we already determined that there can be no overflow, we should add the appropriate NUW/NSW flag.

Diff Detail

Repository
rL LLVM

Event Timeline

nikic created this revision.Apr 11 2019, 2:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 11 2019, 2:18 PM

Looks reasonable.

spatel accepted this revision.Apr 11 2019, 3:53 PM

LGTM

This revision is now accepted and ready to land.Apr 11 2019, 3:53 PM
This revision was automatically updated to reflect the committed changes.