[llvm] r255909 - [ThinLTO] Metadata linking for imported functions

[Teresa Johnson via llvm-commits]

Thanks for pointing this out. I am going through the code now and will fix
these places. There are some on the insert side that could be changed
similarly to insert() rather than count() followed by assignement via
operator[].
Teresa

On Wed, Jan 20, 2016 at 3:31 PM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:

> > +  if (MDValueToValIDMap.count(MD)) {
> > +    unsigned Idx = MDValueToValIDMap[MD];
>
> Hi,
>
> There are many cases like this where we are doing more hash lookups
> than needed. In this case you can use find and compare with end().
>
> Cheers,
> Rafael
>



-- 
Teresa Johnson |  Software Engineer |  tejohnson at google.com |  408-460-2413
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160121/ca64bd76/attachment.html>