[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