Looks good to me.
Looks like we missed a bunch of 'override' in Clang in general, great to put it
in and remove the redundant virtual at the same time.
================
Comment at: include/clang/Analysis/AnalysisContext.h:327
@@ -326,3 +326,3 @@
public:
- ~ScopeContext() {}
+ ~ScopeContext() override {}
----------------
Do we have a clang-tidy warning for explicitly writing default definitions of
functions? Would love to just not have this sort of code written at all.
http://reviews.llvm.org/D8926
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
