[llvm-commits] [llvm] r84533 - /llvm/trunk/lib/Transforms/Scalar/GVN.cpp

Chris Lattner clattner at apple.com
Mon Oct 19 14:24:36 PDT 2009


On Oct 19, 2009, at 2:14 PM, Owen Anderson wrote:

>       if (!local_dep.isDef() && !local_dep.isNonLocal()) {
> -        valueNumbering.insert(std::make_pair(V, nextValueNumber));
> +        valueNumbering[V] =  nextValueNumber;

Note that these have different behavior.  In the former case, if V was  
already in the map, insert is a noop.  In the later case, V gets  
updated.  Please remove the double spaces after the =.

-Chris



More information about the llvm-commits mailing list