[PATCH] D12886: Clean up: Refactoring the hardcoded value of 6 for FindAvailableLoadedValue()'s parameter MaxInstsToScan.

[Larisse Voufo via llvm-commits]

lvoufo added a comment.



> -2-

>  Does "opt -basicaa -gvn" clean up the redundant load? If so, is it okay that instcombine doesn't?


Now it does, as of Diff 35065 of http://reviews.llvm.org/D11826. But it still does not make it okay imo since once might still expect 
something like "-basicaa -instcombine -inline -early-cse -instcombine" to remove redundant loads when 
"-basicaa -instcombine -early-cse -instcombine" does.


http://reviews.llvm.org/D12886