[PATCH] D42180: [NewGVN] Add ops as dependency if we cannot find a leader for ValueOp.
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 18 09:36:20 PDT 2018
On 12/04/2018 17:18, Daniel Berlin wrote:
> Yeah. The tracking is very fine, but is still an overshoot sometimes.
> In these cases, i just want to make sure we are adding the right things
> to track, and not papering over it by just adding a lot of stuff and
> getting the right thing by accident.
Right. This patch adds the translated operands of the
ValueOp which we failed to find a leader for. I think if any of those
change, we might find a new leader and could continue making a PHIOfOps
node. Do you think that's fine-grained enough?
More information about the llvm-commits
mailing list