[PATCH] D128861: [clang-tidy] add cppcoreguidelines-symmetric-binary-operator
Julian Schmidt via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 21 10:54:13 PDT 2023
5chmidti added a comment.
In D128861#4604193 <https://reviews.llvm.org/D128861#4604193>, @PiotrZSL wrote:
> But for me auto-fixes in this check aren't needed, this simply complicate a check that could be very simple.
You might be right. I will fix the issues I have found and will propose this check without the fixes. For completeness, I will mention that I have the implementation for them available, and raise your concern about the complexity of introducing these fixes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128861/new/
https://reviews.llvm.org/D128861
More information about the cfe-commits
mailing list