[llvm] Revert "[Object][WebAssembly] Fix data segment offsets higher than 2^31 (#125739)" (PR #125786)

[Sam Clegg via llvm-commits]

https://github.com/sbc100 created https://github.com/llvm/llvm-project/pull/125786

This reverts commit c798a5c4d5c3c8cb21e6001f505d8f44217c2244.

This broke bunch of test the emscripten side.  Reverting while we investigate.

>From 0ebddb750053ae0599dac0b5aa026549ff753070 Mon Sep 17 00:00:00 2001
From: Sam Clegg <sbc at chromium.org>
Date: Tue, 4 Feb 2025 16:07:09 -0800
Subject: [PATCH] Revert "[Object][WebAssembly] Fix data segment offsets higher
 than 2^31 (#125739)"

This reverts commit c798a5c4d5c3c8cb21e6001f505d8f44217c2244.

This broke bunch of test the emscripten side.  Reverting while we
investigate.
---
 llvm/include/llvm/BinaryFormat/Wasm.h         |  4 ++--
 llvm/lib/Object/WasmObjectFile.cpp            |  4 ++--
 llvm/test/Object/Wasm/data-offsets.yaml       | 22 -------------------
 llvm/test/ObjectYAML/wasm/global_section.yaml |  4 ++--
 .../wasm/invalidate_data_offset.yaml          | 18 ---------------
 5 files changed, 6 insertions(+), 46 deletions(-)
 delete mode 100644 llvm/test/Object/Wasm/data-offsets.yaml
 delete mode 100644 llvm/test/ObjectYAML/wasm/invalidate_data_offset.yaml

diff --git a/llvm/include/llvm/BinaryFormat/Wasm.h b/llvm/include/llvm/BinaryFormat/Wasm.h
index 30271be232ca773..ede2d692a594916 100644
--- a/llvm/include/llvm/BinaryFormat/Wasm.h
+++ b/llvm/include/llvm/BinaryFormat/Wasm.h
@@ -333,8 +333,8 @@ struct WasmTable {
 struct WasmInitExprMVP {
   uint8_t Opcode;
   union {
-    uint32_t Int32;
-    uint64_t Int64;
+    int32_t Int32;
+    int64_t Int64;
     uint32_t Float32;
     uint64_t Float64;
     uint32_t Global;
diff --git a/llvm/lib/Object/WasmObjectFile.cpp b/llvm/lib/Object/WasmObjectFile.cpp
index 7815c2670223198..0f6fd5612f9d82a 100644
--- a/llvm/lib/Object/WasmObjectFile.cpp
+++ b/llvm/lib/Object/WasmObjectFile.cpp
@@ -201,10 +201,10 @@ static Error readInitExpr(wasm::WasmInitExpr &Expr,
   Expr.Inst.Opcode = readOpcode(Ctx);
   switch (Expr.Inst.Opcode) {
   case wasm::WASM_OPCODE_I32_CONST:
-    Expr.Inst.Value.Int32 = readVaruint32(Ctx);
+    Expr.Inst.Value.Int32 = readVarint32(Ctx);
     break;
   case wasm::WASM_OPCODE_I64_CONST:
-    Expr.Inst.Value.Int64 = readVaruint64(Ctx);
+    Expr.Inst.Value.Int64 = readVarint64(Ctx);
     break;
   case wasm::WASM_OPCODE_F32_CONST:
     Expr.Inst.Value.Float32 = readFloat32(Ctx);
diff --git a/llvm/test/Object/Wasm/data-offsets.yaml b/llvm/test/Object/Wasm/data-offsets.yaml
deleted file mode 100644
index b2e1d00675233e4..000000000000000
--- a/llvm/test/Object/Wasm/data-offsets.yaml
+++ /dev/null
@@ -1,22 +0,0 @@
-# RUN: yaml2obj %s | llvm-objdump -s -
-
-## Tests data offsets above 2**31 for I32_CONST and offset over 2**32 work for I64_CONST
-
---- !WASM
-FileHeader:
-  Version:         0x00000001
-Sections:
-  - Type:            DATA
-    Segments:
-      - SectionOffset:   0
-        InitFlags:       0
-        Offset:
-          Opcode:          I32_CONST
-          Value:           2147483649
-        Content:         '6401020304'
-      - SectionOffset:   0
-        InitFlags:       0
-        Offset:
-          Opcode:          I64_CONST
-          Value:           4294967297
-        Content:         '6401020304'
diff --git a/llvm/test/ObjectYAML/wasm/global_section.yaml b/llvm/test/ObjectYAML/wasm/global_section.yaml
index 20f4d5440990fe0..e330b072a726eff 100644
--- a/llvm/test/ObjectYAML/wasm/global_section.yaml
+++ b/llvm/test/ObjectYAML/wasm/global_section.yaml
@@ -10,7 +10,7 @@ Sections:
         Mutable:     false
         InitExpr:
           Opcode:          I64_CONST
-          Value:           5
+          Value:           -5
 ...
 # CHECK: --- !WASM
 # CHECK: FileHeader:
@@ -23,5 +23,5 @@ Sections:
 # CHECK:        Mutable:     false
 # CHECK:        InitExpr:
 # CHECK:          Opcode:          I64_CONST
-# CHECK:          Value:           5
+# CHECK:          Value:           -5
 # CHECK: ...
diff --git a/llvm/test/ObjectYAML/wasm/invalidate_data_offset.yaml b/llvm/test/ObjectYAML/wasm/invalidate_data_offset.yaml
deleted file mode 100644
index ac12026379905ba..000000000000000
--- a/llvm/test/ObjectYAML/wasm/invalidate_data_offset.yaml
+++ /dev/null
@@ -1,18 +0,0 @@
-# RUN: not yaml2obj %s 2>&1 | FileCheck %s
-
-## Tests data offsets above 2**32 are not valid for for I32_CONST
-
---- !WASM
-FileHeader:
-  Version:         0x00000001
-Sections:
-  - Type:            DATA
-    Segments:
-      - SectionOffset:   0
-        InitFlags:       0
-        Offset:
-          Opcode:          I32_CONST
-          Value:           4294967297
-        Content:         '6401020304'
-
-# CHECK: error: out of range number