wabt icon indicating copy to clipboard operation
wabt copied to clipboard

Fix Various TODOs

Open binji opened this issue 5 years ago • 6 comments

There are other TODOs in the source, but most of these should be small.

  • [x] Update output from "type" to "func_type" TODO TODO
  • [ ] Rename LoadStoreExpr (maybe MemoryAccessExpr?) TODO TODO
  • [ ] Use proper location in binary-reader-interp.cc TODO
  • [ ] Implement event matching in interpreter TODO
  • [ ] Clean up "out of bounds" memory/table access messages TODO
  • [ ] Use u16 for SerializedOpcode in interpreter TODO
  • [ ] Move TypeChecker into SharedValidator TODO TODO
  • [ ] Change assert_unlinkable to a single-line error TODO
  • [ ] Print assert_uninstantiable and assert_exhaustion errors when assertion passes TODO TODO
  • [ ] Validate command-line arguments for value-stack-size and call-stack-size TODO TODO TODO TODO
  • [x] Fix typo "initializing" TODO
  • [ ] Remove ParseTextList, switch usage to ParseTextListOpt TODO TODO

binji avatar Mar 25 '20 19:03 binji

Fixed typo "initializing". #1387

Issue-03 avatar Apr 13 '20 08:04 Issue-03

Hey @binji , are most of these items still valid? Would like to pick a few to work on.

raphamorim avatar Mar 19 '23 22:03 raphamorim

My guess is that these are outdated, but I think @sbc100 and @keithw will have more information.

binji avatar Mar 20 '23 16:03 binji

There's still plenty of TODOs in the code (including several of the issues listed here) that seem like fine things to work on.

Other bite-sized-ish priorities could include:

  • Getting simd-everywhere to correctly handle the five SIMD tests where it seems to give results not quite in line with the Wasm spec (https://github.com/WebAssembly/wabt/pull/2021#issuecomment-1396386135). Once we get these passing, wasm2c can claim full SIMD conformance and check the box on webassembly.org, which would be really cool.

  • Fixing some of the oss-fuzz failures (https://bugs.chromium.org/p/oss-fuzz/issues/list?q=wabt).

keithw avatar Mar 21 '23 01:03 keithw

Hey @keithw thanks for the reply, will be taking a look in the SIMD tests.

raphamorim avatar Mar 21 '23 08:03 raphamorim

FWIW, I just submitted a PR to SIMDe that gets the SIMD tests passing on Intel (https://github.com/simd-everywhere/simde/pull/1010)

keithw avatar May 02 '23 09:05 keithw