diff options
| author | Sam Clegg <sbc@chromium.org> | 2019-04-05 00:35:12 +0000 |
|---|---|---|
| committer | Sam Clegg <sbc@chromium.org> | 2019-04-05 00:35:12 +0000 |
| commit | a116d91ba2ad0a648d5359c4f6e79394a51729a6 (patch) | |
| tree | 6140d04f35877cf3c978607b0872d0303a3f8872 /lld | |
| parent | 798e83b5d6853f0c2f68bb4b0785f6a075867a13 (diff) | |
| download | bcm5719-llvm-a116d91ba2ad0a648d5359c4f6e79394a51729a6.tar.gz bcm5719-llvm-a116d91ba2ad0a648d5359c4f6e79394a51729a6.zip | |
[WebAssembly] Include function in wasm table when used in R_WASM_TABLE_INDEX_REL_SLEB
This should have been part of rL357710 but was overlooked because
in our test code the function in question was also used in other
relocations that caused it to be added to the table anyway.
Differential Revision: https://reviews.llvm.org/D60296
llvm-svn: 357737
Diffstat (limited to 'lld')
| -rw-r--r-- | lld/test/wasm/shared.ll | 17 | ||||
| -rw-r--r-- | lld/wasm/Writer.cpp | 3 |
2 files changed, 14 insertions, 6 deletions
diff --git a/lld/test/wasm/shared.ll b/lld/test/wasm/shared.ll index 9d093c42944..40eefe49df0 100644 --- a/lld/test/wasm/shared.ll +++ b/lld/test/wasm/shared.ll @@ -32,6 +32,13 @@ entry: ret i8* bitcast (void ()* @func_external to i8*) } +define default i8* @get_local_func_address() { +entry: + ; Verify that a function which is otherwise not address taken *is* added to + ; the wasm table with referenced via R_WASM_TABLE_INDEX_REL_SLEB + ret i8* bitcast (i8* ()* @get_func_address to i8*) +} + declare void @func_external() ; check for dylink section at start @@ -41,7 +48,7 @@ declare void @func_external() ; CHECK-NEXT: Name: dylink ; CHECK-NEXT: MemorySize: 20 ; CHECK-NEXT: MemoryAlignment: 2 -; CHECK-NEXT: TableSize: 2 +; CHECK-NEXT: TableSize: 3 ; CHECK-NEXT: TableAlignment: 0 ; CHECK-NEXT: Needed: [] ; CHECK-NEXT: - Type: TYPE @@ -61,7 +68,7 @@ declare void @func_external() ; CHECK-NEXT: Table: ; CHECK-NEXT: ElemType: FUNCREF ; CHECK-NEXT: Limits: -; CHECK-NEXT: Initial: 0x00000002 +; CHECK-NEXT: Initial: 0x00000003 ; CHECK-NEXT: - Module: env ; CHECK-NEXT: Field: __stack_pointer ; CHECK-NEXT: Kind: GLOBAL @@ -106,7 +113,7 @@ declare void @func_external() ; CHECK-NEXT: - Offset: ; CHECK-NEXT: Opcode: GLOBAL_GET ; CHECK-NEXT: Index: 2 -; CHECK-NEXT: Functions: [ 3, 0 ] +; CHECK-NEXT: Functions: [ 5, 3, 0 ] ; check the generated code in __wasm_call_ctors and __wasm_apply_relocs functions ; TODO(sbc): Disassemble and verify instructions. @@ -118,7 +125,7 @@ declare void @func_external() ; CHECK-NEXT: Body: 10020B ; CHECK-NEXT: - Index: 2 ; CHECK-NEXT: Locals: [] -; CHECK-NEXT: Body: 230141046A230241006A360200230141086A230241016A3602002301410C6A230141006A360200230141106A23033602000B +; CHECK-NEXT: Body: 230141046A230241016A360200230141086A230241026A3602002301410C6A230141006A360200230141106A23033602000B ; check the data segment initialized with __memory_base global as offset @@ -129,4 +136,4 @@ declare void @func_external() ; CHECK-NEXT: Offset: ; CHECK-NEXT: Opcode: GLOBAL_GET ; CHECK-NEXT: Index: 1 -; CHECK-NEXT: Content: '0200000000000000010000000000000000000000' +; CHECK-NEXT: Content: '0200000001000000020000000000000000000000' diff --git a/lld/wasm/Writer.cpp b/lld/wasm/Writer.cpp index 3450c511831..1af186bcbff 100644 --- a/lld/wasm/Writer.cpp +++ b/lld/wasm/Writer.cpp @@ -1138,7 +1138,8 @@ void Writer::processRelocations(InputChunk *Chunk) { for (const WasmRelocation &Reloc : Chunk->getRelocations()) { switch (Reloc.Type) { case R_WASM_TABLE_INDEX_I32: - case R_WASM_TABLE_INDEX_SLEB: { + case R_WASM_TABLE_INDEX_SLEB: + case R_WASM_TABLE_INDEX_REL_SLEB: { FunctionSymbol *Sym = File->getFunctionSymbol(Reloc.Index); if (Sym->hasTableIndex() || !Sym->hasFunctionIndex()) continue; |
