[PATCH] D90443: [lld][WebAssembly] Give better warnings on bad relocation sites

[Sam Clegg via Phabricator via llvm-commits]

This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG9d1409df8792: [lld][WebAssembly] Give better warnings on bad relocation sites (authored by sbc100).

Changed prior to commit:
  https://reviews.llvm.org/D90443?vs=301806&id=301942#toc

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90443/new/

https://reviews.llvm.org/D90443

Files:
  lld/test/wasm/bad-reloc.yaml
  lld/wasm/InputChunks.cpp


Index: lld/wasm/InputChunks.cpp
===================================================================
--- lld/wasm/InputChunks.cpp
+++ lld/wasm/InputChunks.cpp
@@ -109,9 +109,9 @@
         rel.Type != R_WASM_GLOBAL_INDEX_I32) {
       auto expectedValue = file->calcExpectedValue(rel);
       if (expectedValue != existingValue)
-        warn("unexpected existing value for " + relocTypeToString(rel.Type) +
-             ": existing=" + Twine(existingValue) +
-             " expected=" + Twine(expectedValue));
+        warn(toString(this) + ": unexpected existing value for " +
+             relocTypeToString(rel.Type) + ": existing=" +
+             Twine(existingValue) + " expected=" + Twine(expectedValue));
     }
   }
 }
Index: lld/test/wasm/bad-reloc.yaml
===================================================================
--- /dev/null
+++ lld/test/wasm/bad-reloc.yaml
@@ -0,0 +1,75 @@
+# RUN: yaml2obj %s -o %t.o
+# RUN: not wasm-ld --fatal-warnings -o out.wasm %t.o 2>&1 | FileCheck %s
+#
+# CHECK: bad-reloc.yaml.tmp.o:(_start): unexpected existing value for R_WASM_MEMORY_ADDR_LEB: existing=1 expected=0
+#
+# The code section bellow has been modified such that the target of the
+# relocation is 8180808000 (LEB of 1) rather than 8080808000 (LEB = 0)
+
+--- !WASM
+FileHeader:
+  Version:         0x00000001
+Sections:
+  - Type:            TYPE
+    Signatures:
+      - Index:           0
+        ParamTypes:      []
+        ReturnTypes:
+          - I32
+  - Type:            IMPORT
+    Imports:
+      - Module:          env
+        Field:           __linear_memory
+        Kind:            MEMORY
+        Memory:
+          Initial:         0x00000001
+      - Module:          env
+        Field:           __indirect_function_table
+        Kind:            TABLE
+        Table:
+          Index:           0
+          ElemType:        FUNCREF
+          Limits:
+            Initial:         0x00000000
+  - Type:            FUNCTION
+    FunctionTypes:   [ 0 ]
+  - Type:            DATACOUNT
+    Count:           1
+  - Type:            CODE
+    Relocations:
+      - Type:            R_WASM_MEMORY_ADDR_LEB
+        Index:           1
+        Offset:          0x00000007
+    Functions:
+      - Index:           0
+        Locals:          []
+        Body:            4100280281808080002802000B
+  - Type:            DATA
+    Segments:
+      - SectionOffset:   6
+        InitFlags:       0
+        Offset:
+          Opcode:          I32_CONST
+          Value:           0
+        Content:         '00000000'
+  - Type:            CUSTOM
+    Name:            linking
+    Version:         2
+    SymbolTable:
+      - Index:           0
+        Kind:            FUNCTION
+        Name:            _start
+        Flags:           [ VISIBILITY_HIDDEN ]
+        Function:        0
+      - Index:           1
+        Kind:            DATA
+        Name:            bar
+        Flags:           [ VISIBILITY_HIDDEN ]
+        Segment:         0
+        Size:            4
+    SegmentInfo:
+      - Index:           0
+        Name:            .bss.bar
+        Alignment:       2
+        Flags:           [  ]
+


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90443.301942.patch
Type: text/x-patch
Size: 3163 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201030/1599187b/attachment.bin>