<div dir="ltr">Wouldn't this make sense as a regular warning?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 4:52 AM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.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="">================<br>
Comment at: clang-tidy/misc/CMakeLists.txt:6<br>
@@ -5,2 +5,3 @@<br>
   BoolPointerImplicitConversion.cpp<br>
+  ImplicitArgumentConversion.cpp<br>
   MiscTidyModule.cpp<br>
----------------<br>
</div><div class="">Daniel Jasper wrote:<br>
> ImplicitArgumentConversion is not exactly what this does, I think. Maybe SwappedArgumentsCheck?<br>
</div>+1 for SwappedArgumentsCheck.<br>
<div class=""><br>
================<br>
Comment at: clang-tidy/misc/ImplicitArgumentConversion.cpp:52<br>
@@ +51,3 @@<br>
+  // most likely a swap.<br>
+  if (NumConversions >= 2 && DeclTypes == CallTypes) {<br>
+    diag(Call->getLocStart(),<br>
----------------<br>
</div><div class="">Daniel Jasper wrote:<br>
> Mhmm. This seems like a very crude way of doing this. Might be ok for a first iteration, but it seems like we could actually be much smarter and check if there is a re-ordering that would fit better (and even extend the analysis to more than two parameters, no)?<br>

</div>Instead, I'd check if swapping each consecutive pair of arguments decreases the number of conversions by 2. It would also be nice to specify which pair of arguments is likely to be swapped.<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D4457" target="_blank">http://reviews.llvm.org/D4457</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>