Skip to content

fix(compute): use -gc=precise for TinyGo default build command (#1480) - #1917

Open
Tyagiquamar wants to merge 1 commit into
fastly:mainfrom
Tyagiquamar:fix/tinygo-precise-gc
Open

Tyagiquamar wants to merge 1 commit into
fastly:mainfrom
Tyagiquamar:fix/tinygo-precise-gc

Conversation

@Tyagiquamar

Copy link
Copy Markdown
Contributor

Description

Updates the default fallback build command for TinyGo in \pkg/commands/compute/language_go.go\ from -gc=conservative\ to -gc=precise. The precise collector provides more accurate and efficient garbage collection in Wasm builds compared to conservative GC.

Fixes #1480.

Verification

  • Ran \gofmt -l -w pkg/commands/compute/language_go.go\
  • Ran \go test -v ./pkg/commands/compute/...\ in Docker \golang:1.26-bookworm\ (all tests pass)

@Tyagiquamar
Tyagiquamar requested a review from a team as a code owner September 26, 2026 14:02
@Tyagiquamar
Tyagiquamar requested a review from rcaril September 26, 2026 14:02
@rcaril

rcaril commented Sep 28, 2026

Copy link
Copy Markdown
Member

Hi @Tyagiquamar - please use our PR template:

### Change summary

 <!--
Briefly describe the changes introduced in this pull request. Include context or
reasoning behind the changes, even if they seem minor. If relevant, link to any
related discussions (e.g. Slack threads, tickets, documents).
-->

All Submissions:

* [ ] Have you followed the guidelines in our Contributing document?
* [ ] Have you checked to ensure there aren't other open [Pull Requests](https://github.com/fastly/cli/pulls) for the same update/change?

<!-- You can erase any parts of this template not applicable to your Pull Request. -->

### New Feature Submissions:

* [ ] Does your submission pass tests?

### Changes to Core Features:

* [ ] Have you written new tests for your core changes, as applicable?
* [ ] Have you successfully run tests with your changes locally?

### User Impact

<!-- What is the user impact of this change? -->

### Are there any considerations that need to be addressed for release?

<!-- Any breaking changes, etc -->

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.

TinyGo should use -gc=precise (or have nothing and just use the default provided by tinygo)

2 participants