Skip to content

checker: set ast file fallbacks for vls#25679

Merged
spytheman merged 1 commit intovlang:masterfrom
dy-tea:vls-ice
Nov 7, 2025
Merged

checker: set ast file fallbacks for vls#25679
spytheman merged 1 commit intovlang:masterfrom
dy-tea:vls-ice

Conversation

@dy-tea
Copy link
Member

@dy-tea dy-tea commented Nov 7, 2025

Fixes #25678.

Running the example code with v -vls-mode test.v still causes issues but in an unrelated way.

@spytheman
Copy link
Contributor

Running the example code with v -vls-mode test.v still causes issues but in an unrelated way.

After #25595 , it needs also -check to prevent the cgen stage from running on incomplete data :-| .

@spytheman
Copy link
Contributor

spytheman commented Nov 7, 2025

I think we should either make the builder stop right after the checker, in -vls-mode, or exit inside check_files .

@JalonSolov
Copy link
Collaborator

-vls-mode should never go through the rest of the compiler, so stopping it early would be a good thing.

@spytheman
Copy link
Contributor

Thank you @dy-tea 🙇🏻 .

@spytheman spytheman merged commit 393d5b9 into vlang:master Nov 7, 2025
84 checks passed
@dy-tea dy-tea deleted the vls-ice branch November 7, 2025 19:25
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.

Memory error

3 participants