wasm2c SIMD final approach checklist

[keithw]

wasm2c SIMD final approach checklist

[mr-c]

Maybe controversial: When SIMD is being used, include the simde header in the generated code (like we do with the scalar operations) so the user doesn't need to have a compatible dev repo of SIMDe in their include path to compile the wasm2c output.

This is fine by me, and from the Debian side it would be nice to have a switch to easily use an installed copy of SIMDe instead of the bundled amalgated version.

[keithw]

This is fine by me, and from the Debian side it would be nice to have a switch to easily use an installed copy of SIMDe instead of the bundled amalgated version.

Sounds reasonable... maybe --external-simde or something?

[mr-c]

Sounds reasonable... maybe --external-simde or something?

Oh, I was thinking at compile time of wasm2c output one would use -DEXTERNAL_SIMDE=/usr/include/ or similar

[keithw]

After attempting it, I'm realizing that pasting the simde header into the wasm2c generated code is going to be painful because:

• if the module's public interface has a public-facing v128, or contains a v128 global, we have to include the header in the generated .h file (not just the .c file), and
• the amalgamated header is 17,000 lines

Even if I try to split out the v128 definition into its own header, it still ends up pulling in about 8,000 lines because it wants simde-common.h which wants a bunch of stuff. I'm not seeing an easy way to get simde_v128 robustly without thousands of lines. And I don't really want wasm2c to be pasting all that into a generated .h file. :-(

Maaaybe the better path forward is really just to have an #include <simde/wasm/simd128.h> (in either the .h file or the .c file, depending on whether the public interface includes a v128 or there's a v128 global) and tell the user they'll need to have a compatible version of simde on hand to compile the wasm2c output. :-/

[shravanrn]

Seems reasonable. Based on the above, I suspect us taking ownership of amalgamating simd is going to lead to a bunch of work. Would be happy to include it as part of the output, but it would probably be easier to leave their file structure as is.

[keithw]

Merged in WebAssembly/website#334