[PATCH] D83611: [clang][NFC] Add 'override' keyword to virtual function overrides
Logan Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 10 19:24:42 PDT 2020
logan-5 created this revision.
logan-5 added a project: clang.
Herald added subscribers: cfe-commits, martong.
This patch adds `override` to several overriding virtual functions that were missing the keyword within the clang/ directory. NFC.
These were found by a Clang build equipped with `-Wsuggest-override`, as implemented in this patch <https://reviews.llvm.org/D82728> (D82728 <https://reviews.llvm.org/D82728>).
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D83611
Files:
clang/include/clang/AST/DeclOpenMP.h
clang/lib/AST/Interp/InterpFrame.h
clang/lib/AST/OSLog.cpp
clang/lib/Basic/Targets/OSTargets.h
clang/lib/Sema/SemaDeclCXX.cpp
clang/lib/StaticAnalyzer/Checkers/MacOSKeychainAPIChecker.cpp
clang/lib/StaticAnalyzer/Checkers/NumberObjectConversionChecker.cpp
clang/lib/StaticAnalyzer/Checkers/ObjCContainersChecker.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83611.277200.patch
Type: text/x-patch
Size: 5438 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200711/90a8e8b4/attachment.bin>
More information about the cfe-commits
mailing list