[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