[all-commits] [llvm/llvm-project] e3b79a: [WebAssembly, llvm] Fix buildbot problems with llvm...

[Hood Chatham via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e3b79afa673a029b3b6f546ba59d2998f9cff681
      https://github.com/llvm/llvm-project/commit/e3b79afa673a029b3b6f546ba59d2998f9cff681
  Author: Hood Chatham <roberthoodchatham at gmail.com>
  Date:   2025-07-23 (Wed, 23 Jul 2025)

  Changed paths:
    M llvm/lib/Target/WebAssembly/WebAssemblyISelDAGToDAG.cpp
    M llvm/test/CodeGen/WebAssembly/ref-test-func.ll

  Log Message:
  -----------
  [WebAssembly,llvm] Fix buildbot problems with llvm.wasm.ref.test.func (#150116)

PR #147486 broke the sanitizer and expensive-checks buildbot. 

These captures were needed when toWasmValType emitted a diagnostic but
are no longer needed since we changed it to an assertion failure. This
removes the unneeded captures and should fix the sanitizer-buildbot.

I also fixed the codegen in the wasm64 target: table.get requires an i32
but in wasm64 the function pointer is an i64. We need an additional
`i32.wrap_i64` to convert it. I also added `-verify-machineinstrs` to
the tests so that the test suite validates this fix.

Finally, I noticed that #150201 uses a feature of the intrinsic that is
not covered by the tests, namely `ptr` arguments. So I added one
additional test case to ensure that it works properly.

cc @dschuff



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications