This is an archive of the discontinued LLVM Phabricator instance.

[DSE] Fix bug in partial overwrite tracking
ClosedPublic

Authored by junbuml on Jun 29 2016, 1:11 PM.

Details

Summary

Found cases where DSE incorrectly add partially-overwritten intervals.
Please see the test case for details.

Diff Detail

Event Timeline

junbuml updated this revision to Diff 62273.Jun 29 2016, 1:11 PM
junbuml retitled this revision from to [DSE] Fix bug in partial overwrite tracking.
junbuml updated this object.
junbuml added reviewers: hfinkel, mcrosier, eeckstein.
junbuml added a subscriber: llvm-commits.
hfinkel accepted this revision.Jun 29 2016, 3:39 PM
hfinkel edited edge metadata.

LGTM.

lib/Transforms/Scalar/DeadStoreElimination.cpp
402

There's a missing space in between second and <=. The inner parenthesis are not necessary.

This revision is now accepted and ready to land.Jun 29 2016, 3:39 PM
This revision was automatically updated to reflect the committed changes.