[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