Skip to content

Skip test that fails in STACK_OVERFLOW_CHECK mode. NFC#16821

Merged
sbc100 merged 1 commit intomainfrom
disable_test
Apr 27, 2022
Merged

Skip test that fails in STACK_OVERFLOW_CHECK mode. NFC#16821
sbc100 merged 1 commit intomainfrom
disable_test

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Apr 26, 2022

This is the only test that currently fails in the core2ss mode.

This is the only test that currently fails in the core2ss mode.
@sbc100 sbc100 requested a review from kripken April 26, 2022 22:56
@sbc100 sbc100 enabled auto-merge (squash) April 26, 2022 22:56
Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please open an issue for this if you aren't investigating it atm, this sounds like a possible serious bug.

@sbc100 sbc100 merged commit 63bc62c into main Apr 27, 2022
@sbc100 sbc100 deleted the disable_test branch April 27, 2022 00:44
@sbc100
Copy link
Copy Markdown
Collaborator Author

sbc100 commented Apr 27, 2022

I agree it could be serious, but it seems limited to ASYNCIFY_LAZY_LOAD_CODE.. do you know if that is actually used by folks in the wild?

@kripken
Copy link
Copy Markdown
Member

kripken commented Apr 27, 2022

Good question, I'm not aware of current users. It's an experiment. Seems very low priority then.

We can probably remove lazy-load-code at some point, especially once wasm-split is stable.

sbc100 added a commit that referenced this pull request Apr 27, 2022
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.

2 participants