Page MenuHomeSolus

Update wabt to 1.0.20
ClosedPublic

Authored by YakoYakoYokuYoku on Jan 31 2021, 9:25 PM.
Tags
None
Referenced Files
F11020366: D10479.id25264.diff
Mon, Aug 7, 4:15 AM
F10942013: D10479.id25240.diff
Fri, Jul 14, 8:59 AM
F10898760: D10479.id25264.diff
Jul 3 2023, 5:44 AM
F10894116: D10479.id.diff
Jul 1 2023, 8:24 AM
F10842259: D10479.id25621.diff
Jun 10 2023, 11:04 AM
F10839645: D10479.id25264.diff
Jun 9 2023, 10:09 PM
F10839029: D10479.id25240.diff
Jun 9 2023, 7:05 PM
F10837014: D10479.diff
Jun 8 2023, 10:45 PM
Subscribers

Details

Summary

Changelog:

  • 1.0.20:
    • Rename atomic.notify, *.atomic.wait to memory.atomic.*.
  • 1.0.19:
    • Completely disable signal handler on 32-bit.
  • 1.0.18:
    • Use #warning instead of #warn.

Signed-off-by: Martin Reboredo <yakoyoku@gmail.com>

Test Plan

Run wasm-validate and wasm-strip on a WebAssembly file.

Diff Detail

Repository
R5004 wabt
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

JoshStrobl added inline comments.
pspec_x86_64.xml
62

Have you tested whether or not this could be a shared lib instead of static, and whether or not it needs to be in the main package?

https://github.com/WebAssembly/wabt/blob/master/CMakeLists.txt#L351 might be it.

YakoYakoYokuYoku marked an inline comment as done.

Patch CMakeLists.txt for a wasm-rt-impl shared library.

LGTM but just wanting to confirm that you actually built stuff against this with the change? For example there was a change made to openshadinglanguage that clearly wasn't validated against blender after the changes were made and I ended up needing to fix it while landing it and blender.

Trying to avoid having to do that stuff again :P

emscripten is the only revdep and it's a rundep, also I've retested the commands again with full success.

This revision is now accepted and ready to land.Mar 3 2021, 5:33 PM
This revision was automatically updated to reflect the committed changes.