summaryrefslogtreecommitdiffstats
path: root/build/clang-plugin/TrivialCtorDtorChecker.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'build/clang-plugin/TrivialCtorDtorChecker.cpp')
-rw-r--r--build/clang-plugin/TrivialCtorDtorChecker.cpp29
1 files changed, 29 insertions, 0 deletions
diff --git a/build/clang-plugin/TrivialCtorDtorChecker.cpp b/build/clang-plugin/TrivialCtorDtorChecker.cpp
new file mode 100644
index 0000000000..59576a5b64
--- /dev/null
+++ b/build/clang-plugin/TrivialCtorDtorChecker.cpp
@@ -0,0 +1,29 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#include "TrivialCtorDtorChecker.h"
+#include "CustomMatchers.h"
+
+void TrivialCtorDtorChecker::registerMatchers(MatchFinder *AstMatcher) {
+ AstMatcher->addMatcher(cxxRecordDecl(hasTrivialCtorDtor()).bind("node"),
+ this);
+}
+
+void TrivialCtorDtorChecker::check(const MatchFinder::MatchResult &Result) {
+ const char *Error = "class %0 must have trivial constructors and destructors";
+ const CXXRecordDecl *Node = Result.Nodes.getNodeAs<CXXRecordDecl>("node");
+
+ if (!Node->hasDefinition()) {
+ return;
+ }
+
+ // We need to accept non-constexpr trivial constructors as well. This occurs
+ // when a struct contains pod members, which will not be initialized. As
+ // constexpr values are initialized, the constructor is non-constexpr.
+ bool BadCtor = !(Node->hasConstexprDefaultConstructor() ||
+ Node->hasTrivialDefaultConstructor());
+ bool BadDtor = !Node->hasTrivialDestructor();
+ if (BadCtor || BadDtor)
+ diag(Node->getBeginLoc(), Error, DiagnosticIDs::Error) << Node;
+}