[llvm-commits] [PATCH] Change 'ConstantAggrUniqueMap' to use the new hashing implementation

Jay Foad jay.foad at gmail.com
Tue Mar 6 02:49:59 PST 2012


On 6 March 2012 04:43, Meador Inge <meadori at gmail.com> wrote:
> Hi All,
>
> The attached patch changes 'ConstantAggrUniqueMap' to use the new
> hashing implementation.  I don't love the explicit loops, but
> op_begin/op_end iterate over 'Use's instead of the needed
> 'Constant*'s.  Even with the explicit loops it is a step up form what
> is currently being done.
>
> Full 'make check';  no regressions.  Can someone please commit for me?

Done in r152116. Thanks! (I took the liberty of removing some braces
around a single line for-loop and adjusting the indentation.)

Jay.




More information about the llvm-commits mailing list