Clarification for some settings: are they for Asm.js or Wasm ?

31 views
Skip to first unread message

Gabriel Cuvillier

unread,
Dec 3, 2019, 5:59:33 AM12/3/19
to emscripte...@googlegroups.com
Another technical question for today:

There is a couple of Emscripten settings for which it is not clear if
they have an impact/use on Wasm backend:

- DOUBLE_MODE

- PRECISE_F32

I suppose both of them are only useful for Asm.js, but this is not 100%
clear to me. So I just wanted to know :)

After a second look, it is possible some other settings are also only
relevant for Asm.js (I am thinking about
AGGRESSIVE_VARIABLE_ELIMINATION, INLINING_LIMIT, etc.), and maybe such
info shall be added in the comments (like it have been done recently for
the flags needed for link vs compile+link) ?

Cheers,

Gabriel

Alon Zakai

unread,
Dec 3, 2019, 12:45:44 PM12/3/19
to emscripte...@googlegroups.com
Good idea! I opened


All the ones you mentioned here are fastcomp-only, except for INLINING_LIMIT which definitely should affect closure's inlining, and should also affect LLVM (but we should probably check that, as I'm not sure).


--
You received this message because you are subscribed to the Google Groups "emscripten-discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email to emscripten-disc...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/emscripten-discuss/ff24276d-aed3-83d9-f46c-5d512d3e0aea%40gmail.com.

Shachar Langbeheim

unread,
Dec 3, 2019, 2:09:11 PM12/3/19
to emscripten-discuss
Maybe the options should also be grouped, so that all the fast-comp options will be next to each other, and the same with the LLVM options? It will make the document significantly easier to go over.

Alon Zakai

unread,
Dec 4, 2019, 4:16:15 PM12/4/19
to emscripte...@googlegroups.com
True, however, we'll be removing fastcomp at some point - hopefully not that far off - so maybe a large refactoring of the file would just help for a short time. I'm not opposed to it though.

Reply all
Reply to author
Forward
0 new messages