whisperity added inline comments.
================
Comment at:
clang-tools-extra/test/clang-tidy/checkers/bugprone-easily-swappable-parameters-implicits.cpp:46
+
+void crefToArrayTypedefBoth1(const Point2D &VecDescartes, const Point3D
&VecThreeD) {}
+// NO-WARN: Distinct types.
----------------
aaron.ballman wrote:
> Can you also add a test case like:
> ```
> template <int N, int M>
> void func(int (&array1)[N], int (&array2)[M]) {}
>
> void instantiate() {
> int foo[12], bar[12];
> func(foo, bar);
>
> int baz[12], quux[42];
> func(baz, quux);
> }
> ```
(Implicitly) instantiated decls are excluded from the check as a whole, so this
test case will not drive the check.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106946/new/
https://reviews.llvm.org/D106946
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
