Page MenuHomePhabricator

RegisterCoalescer: Defer clearing implicit_def lanes
Needs ReviewPublic

Authored by arsenm on Thu, Nov 15, 8:58 PM.

Details

Summary

We can't go back and recover the lanes if it turns
out the implicit_def really can't be erased.

Assume all lanes are valid if an unresolved conflict
is encountered. There aren't any tests where this
seems to matter either way, but this seems like a
safer option.

Fixes bug 39602

Diff Detail

Event Timeline

arsenm created this revision.Thu, Nov 15, 8:58 PM
arsenm updated this revision to Diff 174326.Thu, Nov 15, 9:00 PM

Add testcase for the second case where ErasableImplicitDef is cleared, although the used lanes don't matter

arsenm added a reviewer: wmi.Wed, Dec 12, 7:18 PM