<div dir="ltr"><div class="gmail_extra">The fact that the code already existed doesn't mean you should move it into a *much* more widely used location in the codebase without thoroughly understanding it and making sure it works correctly in the generic context. It might have worked fine for a narrow use case but been insufficient for broad use. This seems like a possibility in this case.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">The fundamental point remains -- we need to have unit tests and correct generic abstractions in StringRef. =]</div><div class="gmail_extra"><br><div class="gmail_quote">
On Mon, Jul 8, 2013 at 1:27 PM, Manman Ren <span dir="ltr"><<a href="mailto:mren@apple.com" target="_blank" class="cremed">mren@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The key here is a pair <StringRef and unsigned>. If having a generic DenseMapInfo for StringRef is not a good idea, I will introduce a DenseMapInfo for the pair and keep things where they were by reverting this patch.</blockquote>
</div><br>I think you should do that unless you have solutions to the problems I've described with a generic DenseMapInfo for StringRef (I'm not aware of any, and I think any would require a change to the internals of StringRef which probably isn't desirable).</div>
</div>