impl WasmTy for u32 and u64

[leoyvens]

Wasm integer types are sign agnostic, so if Rust i32 and i64 are WasmTy so should u32 and u64. The trait implentation is exactly the same between i/u32 and i/u64.

[github-actions]

Subscribe to Label Action

cc @peterhuene

Details

This issue or pull request has been labeled: "wasmtime:api"

Thus the following users have been cc'd because of the following labels:

• peterhuene: wasmtime:api

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

[alexcrichton]

Agreed these are good to have, thanks!

Some other items I think to take care of are:

• Could these be written in terms of the i32/i64 impls to cut down on duplication?
• Can documentation be added to Func::wrap about these types?
• Can tests be added which exercise the presence of these impls?

[leoyvens]

@alexcrichton thanks for the review, I've addressed all points.

[alexcrichton]

Looks great to me! As one final nit, there's a table under "Rust types will map to WebAssembly types as follows", mind adding the u32/u64 types there too?

[leoyvens]

@alexcrichton ah of course, I missed that. Updated it.

[alexcrichton]

👍