HomePhabricator

[SimplifyCFG] FoldBranchToCommonDest(): bonus instrns must only be used by PHI…

Authored by lebedev.ri on Dec 12 2020, 4:48 AM.

Description

[SimplifyCFG] FoldBranchToCommonDest(): bonus instrns must only be used by PHI nodes in successors (PR48450)

In particular, if the successor block, which is about to get a new
predecessor block, currently only has a single predecessor,
then the bonus instructions will be directly used within said successor,
which is fine, since the block with bonus instructions dominates that
successor. But once there's a new predecessor, the IR is no longer valid,
and we don't fix it, because we only update PHI nodes.

Which means, the live-out bonus instructions must be exclusively used
by the PHI nodes in successor blocks. So we have to form trivial PHI nodes.
which will then be successfully updated to recieve cloned bonus instns.

This all works fine, except for the fact that we don't have access to
the dominator tree, and we don't ignore unreachable code,
so we sometimes do end up having to deal with some weird IR.

Fixes https://bugs.llvm.org/show_bug.cgi?id=48450

Details

Committed
lebedev.riDec 12 2020, 1:06 PM
Parents
rGce4040a43d54: [PPC] Check for PPC64 when emitting 64bit specific VSX nodes when pattern…
Branches
Unknown
Tags
Unknown