[PATCH] Global Structure Vectorization

Renato Golin renato.golin at linaro.org
Wed Feb 20 14:43:41 PST 2013


On 20 February 2013 19:45, Arnold Schwaighofer <aschwaighofer at apple.com>wrote:

> Take the following as future improvement suggestions, then :)
>

Let me at least try, I think it was just laziness of my part (and the pub).
;)


Since you are in an anonymous namespace (that is, not a public interface)
> you can have the typedef in the enclosing class? (This file already does
> that)
>

Yes, I suggested that earlier, I think I'll go with it.


I still don't see where we would add a loop/complexity in a solution using
> DenseMap<value*, Vector> stead of a multimap without an example :).
>

I think that was before I was using equal_range, I think now you're right,
it shouldn't make *any* difference.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130220/dbe96de9/attachment.html>


More information about the llvm-commits mailing list