<div dir="ltr">UnresolvedSetImpl isn't even constructible, much less copy constructible, but it is a public base class to UnresolvedSet, it it is -- at least until we turn it off by deleting the copy ctor.<div><div><br></div><div><div>template <unsigned InlineCapacity> class UnresolvedSet :</div><div>    public UnresolvedSetImpl {</div><div>  SmallVector<DeclAccessPair, InlineCapacity> Decls;</div><div>};</div></div></div><div><br></div><div>But I'm not sure why we would need to since all it does is contain a SmallVector, and it supports all that behavior.  The problem here is that making UnresolvedSetImpl's ctor private and defining it, means you have to do something with the other ones as well.  I'm happy to turn the all off, but didn't see a good reason to do so.</div><div><br></div><div>As for libcxx and libcxxabi, they are 2 separate repos (at least in git).  Fixing libcxx is easy since it already defines a macro for throw()/noexcept.  Unfortunately, libcxxapi doesn't, though it's easy enough to define.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 14, 2016 at 5:33 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">UNresolvedSetImpl isn't copy constructible, so should it really be copy assignable? (it looks like it only needs to be so because of LookupResult - so once LookupResult is fixed, UnresolvedSetImpl can go back to the way it was)<br><br>Perhaps we should just wait for the fix for LookupResult copying in SemaStmtAsm from Marina?<br><br>Are you planning to cleanup -Wdeprecated in libcxx and libcxxabi too?<br><br>I did most of the -Wdeprecated cleanup late last year - but when I tried to turn it on I hit issues in various stdlibs, etc. It'd be great to finish the cleanup, or suppress it in system headers, etc, and get it turned on.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Mar 14, 2016 at 12:29 PM, don hinton via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">hintonda added a comment.<br>
<br>
Btw, what do you think about making -Wdeprecated the default for llvm/clang builds?<br>
<br>
With this change, llvm/clang will be clean.  However, libcxx and libcxxabi have a bunch of throw warnings.<br>
</div></div><div><div><br>
<br>
<a href="http://reviews.llvm.org/D18123" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18123</a><br>
<br>
<br>
<br><span class="">
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">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>
</span></div></div></blockquote></div><br></div>
</blockquote></div><br></div>