Skip to content

runtime: allow compile-time synctest checks on bare metal - #5974

Open
davecheney wants to merge 1 commit into
tinygo-org:devfrom
davecheney:davecheney-synctest-compile-time-evaluation
Open

davecheney wants to merge 1 commit into
tinygo-org:devfrom
davecheney:davecheney-synctest-compile-time-evaluation

Conversation

@davecheney

@davecheney davecheney commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Let time and channel initialization evaluate synctest checks at compile time on bare-metal targets, avoiding loads the interpreter cannot resolve and reducing firmware size.

Hosted synctest support is unchanged. Starting a synctest bubble on bare metal now panics explicitly.

@davecheney
davecheney requested a review from jakebailey October 21, 2026 00:23
@davecheney

Copy link
Copy Markdown
Contributor Author

@jakebailey mia culpa, this is another example of you adding stdlib support and me going back and trying to compile it out on bare metal. Maybe once #5903 lands the same magic can be applied for the synctest flag.

@davecheney
davecheney force-pushed the davecheney-synctest-compile-time-evaluation branch from 0b4f9c8 to 7539f48 Compare October 21, 2026 03:26
@davecheney davecheney added this to the 0.45 milestone Oct 21, 2026
@davecheney
davecheney force-pushed the davecheney-synctest-compile-time-evaluation branch from 7539f48 to 1b0e5d6 Compare October 22, 2026 13:34

@jakebailey jakebailey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems okay I guess, just not sure how one would ever be in this situation

@davecheney
davecheney force-pushed the davecheney-synctest-compile-time-evaluation branch 5 times, most recently from 683f5f6 to ad1824d Compare October 30, 2026 12:27
Let time and channel initialization evaluate synctest checks at compile time on bare-metal targets, avoiding loads the interpreter cannot resolve and reducing firmware size.

Hosted synctest support is unchanged. Starting a synctest bubble on bare metal now panics explicitly.
@davecheney
davecheney force-pushed the davecheney-synctest-compile-time-evaluation branch 2 times, most recently from 1e91839 to 688fe82 Compare November 1, 2026 07:21
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