<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 14, 2016 at 3:07 PM, don hinton <span dir="ltr"><<a href="mailto:hintonda@gmail.com" target="_blank">hintonda@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 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></blockquote><div><br></div><div>Fair - I'm not too fussed, there, to be sure. If we're going to do it, perhaps we should = default in the move ops too? (except those aren't implicitly defaultible in MSVC... blah)<br><br>Maybe best to just wait until there's a need for them? I'm not sure. *shrug*</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>As for libcxx and libcxxabi, they are 2 separate repos (at least in git).</div></div></blockquote><div><br></div><div>Yep, wasn't expecting the fixes to go in the same code review or anything - just wondering about direction.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>  Fixing libcxx is easy since it already defines a macro for throw()/noexcept.  Unfortunately, libcxxapi doesn't, though it's easy enough to define.<br></div></div></blockquote><div><br></div><div>*nod*</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div></div></div><div class="HOEnZb"><div class="h5"><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>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>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>
_______________________________________________<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>
</div></div></blockquote></div><br></div></div>