Skip to content

Conversation

@sunag
Copy link
Collaborator

@sunag sunag commented Oct 22, 2025

Fixes #30954, #32009,
Related issue: #31929

Description

Variables declared outside the stack are being added to the main stack. This fixes the bug in question, and opens some windows for optimization as well.

@github-actions
Copy link

github-actions bot commented Oct 22, 2025

📦 Bundle size

Full ESM build, minified and gzipped.

Before After Diff
WebGL 354.98
86.28
354.98
86.28
+0 B
+0 B
WebGPU 605.42
169.77
606.15
169.97
+722 B
+200 B
WebGPU Nodes 604.03
169.53
604.75
169.71
+722 B
+180 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Before After Diff
WebGL 486.63
121.03
486.63
121.03
+0 B
+0 B
WebGPU 674.8
185.24
675.25
185.34
+451 B
+105 B
WebGPU Nodes 616.79
168.47
617.25
168.57
+451 B
+102 B

@sunag sunag added this to the r181 milestone Oct 22, 2025
@sunag sunag marked this pull request as ready for review October 28, 2025 13:51
@sunag sunag merged commit 8a74698 into mrdoob:dev Oct 28, 2025
10 checks passed
@sunag sunag deleted the dev-non-stack-var branch October 28, 2025 14:01
Sign up for free to 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.

TSL - Loop with boolean values doesn't work??

1 participant