diff --git a/lld/test/wasm/stack-pointer.ll b/lld/test/wasm/stack-pointer.ll index d37b96eb938a..2eefb2c021ba 100644 --- a/lld/test/wasm/stack-pointer.ll +++ b/lld/test/wasm/stack-pointer.ll @@ -58,6 +58,9 @@ entry: ; CHECK-NEXT: - Name: __wasm_call_ctors ; CHECK-NEXT: Kind: FUNCTION ; CHECK-NEXT: Index: 1 +; CHECK-NEXT: - Name: __stack_pointer +; CHECK-NEXT: Kind: GLOBAL +; CHECK-NEXT: Index: 0 ; CHECK-NEXT: - Name: __heap_base ; CHECK-NEXT: Kind: GLOBAL ; CHECK-NEXT: Index: 1 diff --git a/lld/wasm/Writer.cpp b/lld/wasm/Writer.cpp index 4cefe3a473e4..05bb4c78786e 100644 --- a/lld/wasm/Writer.cpp +++ b/lld/wasm/Writer.cpp @@ -659,9 +659,12 @@ void Writer::calculateExports() { } for (const Symbol *Sym : DefinedGlobals) { - // Can't export the SP right now because it's mutable and mutable globals - // cannot be exported. - if (Sym == Config->StackPointerSymbol) + // Can't export the SP right now because its mutable, and mutuable globals + // are yet supported in the official binary format. However, for + // intermediate output we need to export it in case it is the target of any + // relocations. + // TODO(sbc): Remove this if/when the "mutable global" proposal is accepted. + if (Sym == Config->StackPointerSymbol && !Config->EmitRelocs) continue; ExportedSymbols.emplace_back(WasmExportEntry{Sym, BudgeLocalName(Sym)}); }