Skip to content

Fix too many locals in wasm binary - #1159

Merged
Keavon merged 2 commits into
GraphiteEditor:masterfrom
nat-rix:master
Apr 24, 2023
Merged

Fix too many locals in wasm binary#1159
Keavon merged 2 commits into
GraphiteEditor:masterfrom
nat-rix:master

Conversation

@nat-rix

Copy link
Copy Markdown
Contributor

Split function into smaller non-inlinable ones.
This fixes the issue that rust creates too many wasm locals in non-optimized builds. That lead to a compile error in such builds.

@nat-rix
nat-rixforce-pushed the master branch 2 times, most recently from 1ab1514 to 9fb12afCompareApril 24, 2023 18:09
Comment threadnode-graph/interpreted-executor/src/node_registry.rs Outdated
this fixes the issue that rust creates too many wasm locals in non-optimized
builds. That lead to a compile error in such builds.
@Keavon
Keavon merged commit b0a3632 into GraphiteEditor:masterApr 24, 2023
Keavon added a commit that referenced this pull request Jul 30, 2023
Split function into smaller non-inlinable ones
this fixes the issue that rust creates too many wasm locals in non-optimized
builds. That lead to a compile error in such builds.
Co-authored-by: Keavon Chambers <keavon@keavon.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nat-rix@TrueDoctor@Keavon