]> git.lizzy.rs Git - rust.git/commit
wasm: Increase default stack size to 1MB
authorAlex Crichton <alex@alexcrichton.com>
Thu, 19 Apr 2018 21:51:59 +0000 (14:51 -0700)
committerAlex Crichton <alex@alexcrichton.com>
Thu, 19 Apr 2018 21:51:59 +0000 (14:51 -0700)
commite58629b990ee8ca2dc8fde0703fc7ca2b4b2a809
tree9804eff0ea0b1f9c011f5fb89669d042eaa780a5
parent8830a0304327ba8c983555ac5d42cec0569c31bb
wasm: Increase default stack size to 1MB

This commit increases the dfeault stack size allocated to the
wasm32-unknown-unknown target to 1MB by default. Currently the default stack
size is one wasm page, or 64 kilobytes. This default stack is quite small and
has caused a stack overflow or two in the wild by accident.

The current "best practice" for fixing this is to pass `-Clink-args='-z
stack-size=$bigger'` but that's not great nor always easy to do. A default of
1MB matches more closely with other platforms where it's "pretty big" by
default.

Note that it was tested and if the users uses `-C link-args` to pass a custom
stack size that's still resepected as lld seems to take the first argument, and
where rustc is passing it will always be last.
src/librustc_trans/back/linker.rs