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.
On Tue, Dec 3, 2019 at 11:09 AM Shachar Langbeheim <[email protected]> wrote: > 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. > > On Tue, 3 Dec 2019 at 19:45, Alon Zakai <[email protected]> wrote: > >> Good idea! I opened >> >> https://github.com/emscripten-core/emscripten/pull/9949 >> >> 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). >> >> >> On Tue, Dec 3, 2019 at 2:59 AM Gabriel Cuvillier < >> [email protected]> wrote: >> >>> 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 >>> >>> -- >>> 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 [email protected]. >>> To view this discussion on the web visit >>> https://groups.google.com/d/msgid/emscripten-discuss/ff24276d-aed3-83d9-f46c-5d512d3e0aea%40gmail.com >>> . >>> >> -- >> 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 [email protected]. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/emscripten-discuss/CAEX4NpQFdDdCE71D0vbwAm_R5q_%2B51HetYo1xw0j-qz7oFbjkw%40mail.gmail.com >> <https://groups.google.com/d/msgid/emscripten-discuss/CAEX4NpQFdDdCE71D0vbwAm_R5q_%2B51HetYo1xw0j-qz7oFbjkw%40mail.gmail.com?utm_medium=email&utm_source=footer> >> . >> > -- > 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 [email protected]. > To view this discussion on the web visit > https://groups.google.com/d/msgid/emscripten-discuss/CA%2B_KjGY-JBCNg-8ZKt5OMNtJQP76q5kzHky_SMMXLS_x9fRk8Q%40mail.gmail.com > <https://groups.google.com/d/msgid/emscripten-discuss/CA%2B_KjGY-JBCNg-8ZKt5OMNtJQP76q5kzHky_SMMXLS_x9fRk8Q%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- 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 [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/emscripten-discuss/CAEX4NpREj1RNUTU1%3D1DHNSuBxMrkhQOE5VKqwS1S82Ay07DTOg%40mail.gmail.com.
