PiotrZSL accepted this revision.
PiotrZSL added a comment.
This revision is now accepted and ready to land.
LGTM, but note that type aliases may not work properly, but that more a legacy
issue that got source in line 104, simply because we should use canonical type
there.
================
Comment at:
clang-tools-extra/test/clang-tidy/checkers/readability/non-const-parameter.cpp:235
+ // CHECK-FIXES: {{^}}void recordInitListDiag(const int *x) {{{$}}
+ XYConst xy = {x};
+}
----------------
Add a test with XYConst hidden behind typedef.
like
```typedef XYConst XYConstTypedef;
XYConstTypedef yz = {x};
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158152/new/
https://reviews.llvm.org/D158152
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
