This is an archive of the discontinued LLVM Phabricator instance.

[ADCE] Remove redundent code [NFC]
ClosedPublic

Authored by grosser on Mar 15 2017, 8:42 PM.

Details

Summary

In commit r289548 ([ADCE] Add code to remove dead branches) a redundant loop
nest was accidentally introduced, which implements exactly the same
functionality as has already been available right after. This redundancy has
been found when inspecting the ADCE code in the context of our recent
discussions on post-dominator modeling. This redundant code was also eliminated
by r296535 (which sparked the discussion), but only as part of a larger semantic
change of the post-dominance modeling. As this redundency in [ADCE] is really
just an oversight completely independent of the post-dominance changes under
discussion, we remove this redundancy independently.

Event Timeline

grosser created this revision.Mar 15 2017, 8:42 PM

Hi Daniel,

I pulled this one out, as I believe this will make it easier to understand the impact of the actual post-dom tree.

Btw., thanks for your now very concrete requirement statement. I will reply to the thread soonish.

This revision is now accepted and ready to land.Mar 15 2017, 9:04 PM
This revision was automatically updated to reflect the committed changes.