[PATCH] D42180: [NewGVN] Add ops as dependency if we cannot find a leader for ValueOp.

[Florian Hahn via llvm-commits]

On 12/04/2018 16:48, Daniel Berlin wrote:
> Sorry, i'll stare at this again.
> 
> 

Great thanks.

I suppose this still leaves a few unnecessary users around in some 
cases, e.g. if we add additional users for `I` because we failed to find 
a leader for a ValueOp. When we later find a leader for that ValueOp, 
the other additional users are still stuck around, even though we only 
need them if we managed to make a phiOfOps.