[PATCH] D81737: docker images for mlir-nvidia

Christian Kühnel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 16 02:09:32 PDT 2020


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
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81737.270984.patch
Type: text/x-patch
Size: 866 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200616/9e04ee39/attachment-0001.bin>


More information about the cfe-commits mailing list