[PATCH] D129570: Add new clang-tidy check to find implicit conversions from enum to integer.

2022-08-10 Thread Paul Fultz II via Phabricator via cfe-commits
pfultz2 updated this revision to Diff 451666. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129570/new/ https://reviews.llvm.org/D129570 Files: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp clang-tools-extra/clang-tidy/bugprone/CMakeLists.txt

[PATCH] D129570: Add new clang-tidy check to find implicit conversions from enum to integer.

2022-08-09 Thread Artem Tamazov via Phabricator via cfe-commits
artem.tamazov accepted this revision. artem.tamazov added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129570/new/ https://reviews.llvm.org/D129570

[PATCH] D129570: Add new clang-tidy check to find implicit conversions from enum to integer.

2022-08-09 Thread Paul Fultz II via Phabricator via cfe-commits
pfultz2 updated this revision to Diff 451284. pfultz2 added a comment. Fix typo and merge from main. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129570/new/ https://reviews.llvm.org/D129570 Files:

[PATCH] D129570: Add new clang-tidy check to find implicit conversions from enum to integer.

2022-07-25 Thread Artem Tamazov via Phabricator via cfe-commits
artem.tamazov requested changes to this revision. artem.tamazov added a comment. This revision now requires changes to proceed. A typo in the doc, but otherwise LGTM. Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone/enum-to-int.rst:8 +integer. In C++11, enums can

[PATCH] D129570: Add new clang-tidy check to find implicit conversions from enum to integer.

2022-07-12 Thread Paul Fultz II via Phabricator via cfe-commits
pfultz2 created this revision. pfultz2 added reviewers: aaron.ballman, alexfh. pfultz2 added a project: clang-tools-extra. Herald added subscribers: carlosgalvezp, mgorny. Herald added a project: All. pfultz2 requested review of this revision. Herald added a subscriber: cfe-commits. This check