Page MenuHomePhabricator

[SystemZ] Fix addcarry of addcarry of const carry (PR42606)

Authored by nikic on Fri, Jul 12, 12:16 PM.



This fixes by extending D64213. Instead of only checking if the carry comes from a matching operation, we now check the full chain of carries. Otherwise we might custom lower the outermost addcarry, but then generically legalize an inner addcarry.

This feels a bit hacky...

Diff Detail


Event Timeline

nikic created this revision.Fri, Jul 12, 12:16 PM
Herald added a project: Restricted Project. · View Herald TranscriptFri, Jul 12, 12:16 PM
uweigand accepted this revision.Fri, Jul 12, 12:27 PM

I agree it's a bit hacky, but I don't see any better solution either. In any case, the change is not wrong -- everything rejected should be rejected. Given that this fixes a real problem, I'm fine with the patch. LGTM.

This revision is now accepted and ready to land.Fri, Jul 12, 12:27 PM
This revision was automatically updated to reflect the committed changes.