[PATCH] D42028: [WebAssembly] Fix build failures due to warning

Nicholas Wilson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 13 03:46:28 PST 2018


ncw created this revision.
ncw added reviewers: sbc100, ruiu.
Herald added subscribers: llvm-commits, sunfish, aheejin, jgravelle-google, dschuff, jfb.

The LLD waterfall is broken!

Oops, the waterfall tests with `-Werror -Wunused-variable`.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42028

Files:
  wasm/Writer.cpp


Index: wasm/Writer.cpp
===================================================================
--- wasm/Writer.cpp
+++ wasm/Writer.cpp
@@ -443,8 +443,10 @@
     if (InputSegments.empty())
       continue;
     StringRef Comdat = InputSegments[0]->getComdat();
-    for (const InputSegment *IS : InputSegments)
+    for (const InputSegment *IS : InputSegments) {
+      (void)IS;
       assert(IS->getComdat() == Comdat);
+    }
     if (!Comdat.empty())
       Comdats[Comdat].emplace_back(ComdatEntry{WASM_COMDAT_DATA, I});
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42028.129757.patch
Type: text/x-patch
Size: 526 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180113/f12d32fe/attachment.bin>


More information about the llvm-commits mailing list