Page MenuHomePhabricator

TailDuplicator: Remove no-op analyzeBranch call
ClosedPublic

Authored by arsenm on Jun 6 2019, 8:14 AM.

Details

Reviewers
iteratee
sanjoy
Summary

This could fail, which looked concerning. However nothing was actually
using the results of this. I assume this was intended to use the
anti-feature of analyzeBranch of removing instructions, but wasn't
actually calling it with AllowModify = true.

Fixes bug 42162.

Diff Detail

Event Timeline

arsenm created this revision.Jun 6 2019, 8:14 AM
sanjoy accepted this revision.Jun 6 2019, 7:59 PM

lgtm

This revision is now accepted and ready to land.Jun 6 2019, 7:59 PM
arsenm closed this revision.Jun 7 2019, 6:30 AM

r362800