Page MenuHomePhabricator

LiveIntervals: Remove assertion
ClosedPublic

Authored by arsenm on Sep 12 2019, 9:06 AM.

Details

Reviewers
qcolombet
MatzeB
Summary

This testcase is invalid, and caught by the verifier. For the verifier
to catch it, the live interval computation needs to complete. Remove
the assert so the verifier catches this, which is less confusing.

In this testcase there is an undefined use of a subregister, and lanes
which aren't used or defined. An equivalent testcase with the
super-register shrunk to have no untouched lanes already hit this
verifier error.

Diff Detail

Event Timeline

arsenm created this revision.Sep 12 2019, 9:06 AM
This revision is now accepted and ready to land.Sep 12 2019, 10:25 AM
arsenm closed this revision.Sep 12 2019, 4:52 PM

r371792