Skip to content

Commit 87828ee

Browse files
committed
Cleanup TODO
1 parent 652ff3f commit 87828ee

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

TODO.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,16 @@
11
# Critical Priority
22

33
- Replace `llvm-mc` with `wasm-tools`. See [bytecodealliance/wasm-tools#2405].
4+
- Compatibility with `wasm-bindgen`.
45
- Do we agree with the new name `js-bindgen`? Maybe `web-bindgen` is the most accurate?
56
- `#[js_sys]` is probably also not the ideal name.
67

78
[bytecodealliance/wasm-tools#2405]: https://github.com/bytecodealliance/wasm-tools/issues/2405
89

910
# High Priority
1011

12+
- Add a `disable_hygiene` attribute to `#[js_sys]` to reduce the compile-time of the output to an
13+
absolute minimum. This can avoid all `interpolate`s.
1114
- Escape namespaces and function names if they are not valid JS identifiers.
1215
- Test raw Rust identifiers in function and parameter names.
1316
- `#[js_sys]` parameter and function name aren't allowed to collide. Internal
@@ -33,17 +36,13 @@
3336

3437
# Medium Priority
3538

36-
- Find a way to link to imports directly if no wrapper function is needed. This happens no
39+
- Find a way to link to imports directly if no wrapper function is needed. This happens when no
3740
conversions on the ASM level are necessary.
3841
- Provide an absolutely minimal allocator.
3942
- The `js_sys` proc-macro should remove the `extern "C" { ... }` part of the input on error to avoid
4043
triggering the `unsafe` requirement downstream.
4144
- Optimize linker file interactions by using memory mapped files instead of reading and writing
4245
everything into memory.
43-
- Run the assembly compiler on the proc-macro level so users see errors without having to engage the
44-
linker.
45-
- Parse the JS on the proc-macro level so users see errors. E.g. `oxc-parser`.
46-
- Use an AST for JS code generation so we don't play around with strings. E.g. `oxc-codegen`.
4746

4847
[Emscripten's]:
4948
https://github.com/emscripten-core/emscripten/blob/28bcb86466a273859b8adb43cb167b97e05e145d/src/lib/libstrings.js

0 commit comments

Comments
 (0)