This is an archive of the discontinued LLVM Phabricator instance.

Paths

[WebAssembly] Fix build failures due to warning
ClosedPublic

Authored by ncw on Jan 13 2018, 3:46 AM.

Reviewers

sbc100
ruiu

Commits

Summary

The LLD waterfall is broken!

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

This revision was not accepted when it landed; it landed in state Needs Review.Jan 13 2018, 8:01 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

lld/

trunk/

wasm/

2 lines

Show First 20 Lines • Show All 437 Lines • ▼ Show 20 Lines	if (!Comdat.empty())
Comdats[Comdat].emplace_back(		Comdats[Comdat].emplace_back(
ComdatEntry{WASM_COMDAT_FUNCTION, F->getOutputIndex()});		ComdatEntry{WASM_COMDAT_FUNCTION, F->getOutputIndex()});
}		}
for (uint32_t I = 0; I < Segments.size(); ++I) {		for (uint32_t I = 0; I < Segments.size(); ++I) {
const auto &InputSegments = Segments[I]->InputSegments;		const auto &InputSegments = Segments[I]->InputSegments;
if (InputSegments.empty())		if (InputSegments.empty())
continue;		continue;
StringRef Comdat = InputSegments[0]->getComdat();		StringRef Comdat = InputSegments[0]->getComdat();
		#ifndef NDEBUG
for (const InputSegment *IS : InputSegments)		for (const InputSegment *IS : InputSegments)
assert(IS->getComdat() == Comdat);		assert(IS->getComdat() == Comdat);
		#endif
if (!Comdat.empty())		if (!Comdat.empty())
Comdats[Comdat].emplace_back(ComdatEntry{WASM_COMDAT_DATA, I});		Comdats[Comdat].emplace_back(ComdatEntry{WASM_COMDAT_DATA, I});
}		}

if (!Comdats.empty()) {		if (!Comdats.empty()) {
SubSection SubSection(WASM_COMDAT_INFO);		SubSection SubSection(WASM_COMDAT_INFO);
writeUleb128(SubSection.getStream(), Comdats.size(), "num comdats");		writeUleb128(SubSection.getStream(), Comdats.size(), "num comdats");
for (const auto &C : Comdats) {		for (const auto &C : Comdats) {
▲ Show 20 Lines • Show All 417 Lines • Show Last 20 Lines