Page MenuHomePhabricator

[Statepoint] Handle 'undef' operand tied to def
ClosedPublic

Authored by skatkov on Feb 2 2021, 2:18 AM.

Details

Summary

FixupStatepoints pass does not take into account the undef use
it skips may have a tied def. So when defs are handled pass
considers that tied-use should be spilled and triggers an assert.

FixupStatepoints should skip undef def as well.

Diff Detail

Event Timeline

skatkov created this revision.Feb 2 2021, 2:18 AM
skatkov requested review of this revision.Feb 2 2021, 2:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 2 2021, 2:18 AM
dantrushin accepted this revision.Feb 2 2021, 3:46 AM

LGTM.

BTW, we can completely delete tied dead,undef pair. But additional complexity might not worth it.

This revision is now accepted and ready to land.Feb 2 2021, 3:46 AM
This revision was automatically updated to reflect the committed changes.