[PATCH] Store intrinsic ID by value in Function instead of a string lookup. NFC

Pete Cooper peter_cooper at apple.com
Mon May 18 17:29:42 PDT 2015


> On May 18, 2015, at 5:05 PM, Reid Kleckner <rnk at google.com> wrote:
> 
> You are a hero for doing this. :) I was looking at this the other day and thinking "really, a DenseMap to look up a 32-bit integer?”
Thanks :)  It was pretty scary looking code for something we call so frequently.

BTW, the commits ended up being r237641 and r237642.

Cheers,
Pete
> 
> http://reviews.llvm.org/D9836
> 
> EMAIL PREFERENCES
>  http://reviews.llvm.org/settings/panel/emailpreferences/
> 
> 





More information about the llvm-commits mailing list