hokein added a comment.

> This check looks like specific case of https://reviews.llvm.org/D23427. May 
> be they should be merged?


+1, I think this check can be merged there.


================
Comment at: test/clang-tidy/misc-comparison-function-address.cpp:20
@@ +19,3 @@
+  // CHECK-MESSAGES: :[[@LINE-1]]:12: warning: Address of function is compared 
[misc-comparison-function-address]
+  
+  if (0 == getc);
----------------
How about the case `if (getc == func)`?


Repository:
  rL LLVM

https://reviews.llvm.org/D23423



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to