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:
Eugene.Zelenko added a comment.
This check looks like specific case of https://reviews.llvm.org/D23427. May be
they should be merged?
Repository:
rL LLVM
https://reviews.llvm.org/D23423
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
Eugene.Zelenko added a comment.
Clang-tidy terminology use //checks//, not //checkers//.
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/cf
Eugene.Zelenko added a comment.
Please mention this check in docs/ReleaseNotes.rst (in alphabetical order).
If I'm not mistaken, GCC or Clang has warning that result of comparison is
always true for such situations.
Comment at: clang-tidy/misc/ComparisonFunctionAddressCheck.h:
