[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