[PATCH] D147924: [clang-tidy] Exclude template instantiations in modernize-use-override

Piotr Zegar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 10 02:37:55 PDT 2023


PiotrZSL created this revision.
PiotrZSL added reviewers: njames93, carlosgalvezp.
Herald added a subscriber: xazax.hun.
Herald added a project: All.
PiotrZSL requested review of this revision.
Herald added a project: clang-tools-extra.
Herald added a subscriber: cfe-commits.

Fixed false positive in modernize-use-override check that occurred
when warnings were generated for template classes that were valid
for some instantiations but potentially invalid for others.
Template instantiations are now excluded from the check.

Fixes #38276.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D147924

Files:
  clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp
  clang-tools-extra/docs/ReleaseNotes.rst
  clang-tools-extra/test/clang-tidy/checkers/modernize/use-override.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147924.512101.patch
Type: text/x-patch
Size: 4936 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230410/2f6b926b/attachment.bin>


More information about the cfe-commits mailing list