[PATCH] D37359: [WebAssembly] Fix getSymbolValue() for data symbols

don hinton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 3 22:18:19 PDT 2017


hintonda added inline comments.


================
Comment at: llvm/trunk/lib/Object/WasmObjectFile.cpp:794
+    const wasm::WasmGlobal& Global = Globals[GlobalIndex];
+    return Global.InitExpr.Value.Int32;
+  }
----------------
This is causing buildbot errors on big endian machines when the Opcode is wasm::WASM_OPCODE_I64_CONST, which corresponds to Int64, not Int32, e.g.: 

  http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/7421



Repository:
  rL LLVM

https://reviews.llvm.org/D37359





More information about the llvm-commits mailing list