<div dir="ltr"><span style="font-size:12.8px">> Can we hold off a little longer?</span><br><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Yes we can. The entire point of the AFAIK was to encourage migration to libc++ so it makes some</span></div><div><span style="font-size:12.8px">sense to keep them while they still serve this purpose.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">When you say "a little longer" what sort of timeline do you have in mind?</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">It would be nice to get rid of these in the *long term*, but they aren't in danger ATM.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">/Eric</span></div><div><span style="font-size:12.8px"><br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 10, 2016 at 3:17 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
> On 2016-Feb-10, at 13:25, Bob Wilson <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:<br>
><br>
><br>
>> On Feb 10, 2016, at 12:59 PM, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br>
>><br>
>> On 10 February 2016 at 12:52, Eric Fiselier <<a href="mailto:eric@efcs.ca">eric@efcs.ca</a>> wrote:<br>
>>> @Tim Are these tests in the clang test suite?<br>
>><br>
>> Yep, at <a href="http://llvm.org/git/test-suite.git" rel="noreferrer" target="_blank">http://llvm.org/git/test-suite.git</a>.<br>
>><br>
>>> Marshall and I were just talking about removing <ext/hash_map> all together. Could you explain who still uses it?<br>
>><br>
>> In the test-suite, it looks like it's just these two micro-benchmarks.<br>
>> I doubt anyone would complain about converting them to use<br>
>> std::unordered_map (especially since the underlying implementation<br>
>> seems to be shared anyway).<br>
>><br>
>> Outside that, we seem to have a handful of internal users and Bob may<br>
>> have ABI backwards-compatibility concerns (or not). I've added him<br>
>> explicitly.<br>
><br>
> I will defer to Duncan on that question.<br>
<br>
</div></div>Removing ext/hash_map and ext/hash_set is a great idea.  I don't have<br>
any ABI concerns with removing them, since they are both header-only.<br>
<br>
Unfortunately, we still have users on libstdc++ that use ext/hash_set<br>
and ext/hash_map, and deleting these would make migration to libc++<br>
harder.  Can we hold off a little longer?  Feel free to file a PR and<br>
assign it to me.</blockquote></div><br></div>