<p dir="ltr">Great! I think we should proceed with your is_trivially_copyable idea and I'll see if I can come up with a pathological test case that breaks it.</p>
<p dir="ltr">Hopefully we can find something that works.</p>
<div class="gmail_quote">On Mar 17, 2016 2:54 PM, "Duncan P. N. Exon Smith via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">r263746.<br>
<br>
> On 2016-Mar-17, at 13:36, Duncan P. N. Exon Smith via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
><br>
>><br>
>> On 2016-Mar-16, at 15:41, Eric Fiselier <<a href="mailto:eric@efcs.ca">eric@efcs.ca</a>> wrote:<br>
>><br>
>> EricWF accepted this revision.<br>
>> EricWF added a comment.<br>
>> This revision is now accepted and ready to land.<br>
>><br>
>> LGTM after change in inline comment.<br>
>><br>
>><br>
>> ================<br>
>> Comment at: include/__hash_table:114<br>
>> @@ +113,3 @@<br>
>> +template <class _Pair, class _Key, class _First, class _Second><br>
>> +struct __can_extract_key<_Pair, _Key, pair<_First, _Second>><br>
>> +    : conditional<is_same<typename remove_const<_First>::type, _Key>::value,<br>
>> ----------------<br>
>> `>>` needs to be `> >` for C++03.<br>
><br>
> Since __can_extract_key is only used when !defined(_LIBCPP_CXX03_LANG),<br>
> I'll just move it behind an #ifdef (and leave the prettier `>>`).<br>
><br>
> I'll commit after running the tests with -std=c++03.<br>
><br>
>><br>
>> <a href="http://reviews.llvm.org/D16360" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16360</a><br>
>><br>
>><br>
>><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>