[llvm-commits] PATCH: Switch StringMap to the new hashing infrastructure

Chandler Carruth chandlerc at google.com
Thu Mar 8 16:44:15 PST 2012


On Thu, Mar 8, 2012 at 4:30 PM, Chris Lattner <clattner at apple.com> wrote:

> Ok.  Thanks for being methodical!


For what its worth, I'm going to get the code-size down before I move
forward with committing this. The optimizer should be doing a bit better
job here. ;] There are lots of places where we lookup constant-sized
entities in a string map, and for those cases, we don't emit very good code
even if they aren't (at the moment) hot.

I'll re-run my experiment once I've done the best I can there, and commit
if the numbers are at least no worse. I don't expect any change in run-time
though, as it seems that the hot path doesn't currently overlap with the
poorly optimized path.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120308/85558e65/attachment.html>


More information about the llvm-commits mailing list