kuhnel updated this revision to Diff 270984.
kuhnel marked 3 inline comments as done.
kuhnel added a comment.
Herald added subscribers: cfe-commits, martong.
Herald added a project: clang.
improved tagging of images
Repository:
rZORG LLVM Github Zorg
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81737/new/
https://reviews.llvm.org/D81737
Files:
clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
Index: clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
===================================================================
--- clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
+++ clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
@@ -273,7 +273,7 @@
resolveDependencies<true>();
resolveDependencies<false>();
-#ifndef NDEBUG // avoid -Wunused warnings in release build.
+#ifndef NDEBUG
for (auto &DepPair : Dependencies) {
for (auto &WeakDepPair : WeakDependencies) {
// Some assertions to enforce that strong dependencies are relations in
@@ -285,9 +285,7 @@
"A strong dependency mustn't have weak dependencies!");
assert(WeakDepPair.second != DepPair.second &&
"A strong dependency mustn't be a weak dependency as well!");
- (void)WeakDepPair;
}
- (void)DepPair;
}
#endif
Index: clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
===================================================================
--- clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
+++ clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
@@ -273,7 +273,7 @@
resolveDependencies<true>();
resolveDependencies<false>();
-#ifndef NDEBUG // avoid -Wunused warnings in release build.
+#ifndef NDEBUG
for (auto &DepPair : Dependencies) {
for (auto &WeakDepPair : WeakDependencies) {
// Some assertions to enforce that strong dependencies are relations in
@@ -285,9 +285,7 @@
"A strong dependency mustn't have weak dependencies!");
assert(WeakDepPair.second != DepPair.second &&
"A strong dependency mustn't be a weak dependency as well!");
- (void)WeakDepPair;
}
- (void)DepPair;
}
#endif
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
