Page MenuHomePhabricator

[NewGVN] Add phi-of-ops instr as user of FoundVal.
Needs ReviewPublic

Authored by fhahn on Aug 28 2019, 9:49 PM.

Details

Reviewers
efriedma
davide
Summary

If any of the values we chose as translated PHI operands
change, we have to re-evaluate the phi-of-ops for
I. We already record additional dependencies for
the phi translated operands, but we miss the link
to the translated operand. Those can change without
without their operands changing, e.g. if the leader
of their congruence class changes.

Even with this patch, there are still some dependencies
missing. I will continue tackling the outstanding
reporeted crashes in this area.

Fixes PR36501, PR42422, PR42557.

Event Timeline

fhahn created this revision.Aug 28 2019, 9:49 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2019, 9:49 PM