Alon Zakai

> For a bit of background: What I previously did whenever I wanted to quickly check if an expression evaluates to a constant during codegen was to create a temporary...

> a way to tell the precompute pass that it can ignore side effects if only evaluating, but not going to replace. That sounds good to me too. Basically "compute...

Number of hints. So if function analysis gives 100 hints in the entire module, cross-function analysis adds 3-50 on top.

Update: * `isDebugEnabled` that is mentioned before is no longer a performance bottleneck, after some optimization work. * `BYN_ENABLE_ASSERTIONS=OFF` helps very little with speed - I see just a 2%...

Yes, `sdk-upstream-main-64bit` are for compiling from source, I think those are still valid. I opened https://github.com/emscripten-core/emscripten/pull/24484 to add docs for the asserts builds.

Hmm, this PR fails many tests because of these TODOs for typed continuations: https://github.com/WebAssembly/binaryen/blob/f8531ef56e61cb68815c835b294777fbd97f3b0a/src/ir/subtype-exprs.h#L415-L423 This problem becomes noticeable in this PR because we have tests that use `-all` on continuations...

Thanks, I don't think I'd have time today myself.

If you do not specify optimizations, they will not run - what optimizations did you see happen that you didn't want? (Note that wasm2js specifically does run some passes internally,...

Oh, it sounds like you want to build Binaryen itself without optimizations enabled? There isn't an option for that. You could probably hack up `src/passes/CMakeLists, pass.cpp` to remove passes you...

@fgmccabe Hmm, is that part not specifically for Name annotations? It is in the "Name Annotations" subsection. (Though maybe it suggests a general rule?)