================

----------------
5chmidti wrote:

Thanks for changing these to be more understandable w.r.t. what has changed. 
Could you please do a follow-up PR that splits this file, so we have the diff 
that shows no capabilities were lost in this PR/commit, and then have the tests 
be where the checks are (e.g., 
`misc/redundant-expression-csa-identicalexpr.cpp`?)

---

FYI: When merging with the other tests in a follow-up, note that clang-tidy has 
a test-driver for lit, e.g., 
- `// RUN: %check_clang_tidy %s -check-suffixes=,DEFAULT 
cppcoreguidelines-avoid-non-const-global-variables %t` for `CHECK-MESSAGES` and 
`CHECK-MESSAGES-DEFAULT`
- `// RUN: %check_clang_tidy %s 
cppcoreguidelines-avoid-non-const-global-variables %t` for `CHECK-MESSAGES`

https://github.com/llvm/llvm-project/pull/114715
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to