[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:23:52 PDT 2023


5chmidti abandoned this revision.
5chmidti added a comment.

Because there was no activity on this patch, I am closing it. I should've pinged a bit more :) 
With the move to GitHub pull requests soon, pushing this patch further in phabricator does not make sense to me. Instead, I could reopen this patch as a pull request. There are some problems I have encountered anyway that need fixing.
Is there general interest in this check?
The cppcoreguidelines prefix could be changed to something else (e.g. modernize), if that is a hindrance.


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