<div dir="ltr">I don't feel sufficiently strongly to insist - clang-tidy's mostly outside my wheelhouse anyway.<br><br>As for how to go about it - my rough approach would be to write a small test case that calls an implicitly-deleted-but-explicitly-defaulted move op, run it, check the diagnostic text, find that in DiagnosticSemaKinds, find where the diagnostic identifier is used in the code, set a breakpoint, break there - then trace back to when/where it got marked as deleted (find the member function of the AST node in question that is used to mark it as deleted - break in there, with the condition that 'this ==' the specific instance you're looking at, etc). Then see if, at the point at which the thing is marked as deleted, you can check that it's not dependent (not a template, etc) and emit the appropriate warning.<br><br>- Dave</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 11, 2016 at 9:44 AM, Alex Pilkiewicz 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><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">pilki added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D18961#397163" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18961#397163</a>, @dblaikie wrote:<br>
<br>
> I'd consider just making this a compiler warning, perhaps?<br>
<br>
<br>
</span>I honestly don't feel competent right now to write compiler warnings: clang-tidy has a nice, well defined interface. A compiler warning would force me to dig in the internals of clang. But if you feel strongly about it and give me a couple of pointers (like a diff that adds a similar warning, and maybe a pointer to the general area in the code where I would add it), I can maybe give it a try in the next few days/weeks.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D18961" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18961</a><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>
</div></div></blockquote></div><br></div>