From a38ed62ea803a41c2e1f3fa12edbc176e474056f Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Tue, 1 Dec 2020 14:04:21 -0800 Subject: [PATCH] [lld][WebAssembly] Feedback from D92038. NFC Differential Revision: https://reviews.llvm.org/D92429 --- lld/test/wasm/weak-undefined-pic.s | 4 ++-- lld/wasm/Symbols.h | 3 +++ 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/lld/test/wasm/weak-undefined-pic.s b/lld/test/wasm/weak-undefined-pic.s index c12ef23..a169fd3 100644 --- a/lld/test/wasm/weak-undefined-pic.s +++ b/lld/test/wasm/weak-undefined-pic.s @@ -27,7 +27,7 @@ _start: .weak foo .functype foo () -> (i32) -# Verify that we do not generate dynamnic relocations for the GOT entry. +# Verify that we do not generate dynamic relocations for the GOT entry. # CHECK-NOT: __wasm_apply_relocs @@ -68,7 +68,7 @@ _start: # CHECK-NEXT: - Index: 1 # CHECK-NEXT: Name: 'undefined_weak:foo' -# With `-pie` or `-shared` the resolution should is defered to the dynamic +# With `-pie` or `-shared` the resolution should be deferred to the dynamic # linker and the function address should be imported as GOT.func.foo. # # RUN: wasm-ld --experimental-pic -pie %t.o -o %t3.wasm diff --git a/lld/wasm/Symbols.h b/lld/wasm/Symbols.h index 90fb519..cfa6869 100644 --- a/lld/wasm/Symbols.h +++ b/lld/wasm/Symbols.h @@ -160,6 +160,9 @@ public: // True if this symbol is a linker-synthesized stub function (traps when // called) and should otherwise be treated as missing/undefined. See // SymbolTable::replaceWithUndefined. + // These stubs never appear in the table and any table index relocations + // against them will produce address 0 (The table index representing + // the null function pointer). bool isStub : 1; uint32_t flags; -- 2.7.4