Conversation
Owner
Author
|
CI triage: both current failures occur before this PR's code runs.
No artifact-lifecycle change is indicated by either log. |
cpunion
force-pushed
the
codex/wasm-r3-artifact-lifecycle-20260905
branch
from
September 5, 2026 14:47
4a36fa0 to
f06850e
Compare
cpunion
force-pushed
the
codex/wasm-r3-runner-diagnostics-20260905
branch
from
September 5, 2026 14:47
561e950 to
2be342a
Compare
cpunion
force-pushed
the
codex/wasm-r3-artifact-lifecycle-20260905
branch
from
September 5, 2026 15:22
f06850e to
5435e47
Compare
cpunion
force-pushed
the
codex/wasm-r3-runner-diagnostics-20260905
branch
from
September 5, 2026 15:22
2be342a to
946f93e
Compare
Owner
Author
|
Rebased in dependency order onto LLVM 22 main
To keep CI load bounded, this intermediate PR retains Format/Targets. Repeated functional matrices are canceled here and kept on the updated stack tip #235. Per-layer benchmark reruns are deferred, not replaced by the top PR's performance comparison. Canceled checks are not passing checks, and the integration result is not independent complete validation of every intermediate revision. |
Owner
Author
|
Retiring this old R3 stack while its still-useful changes are extracted into a new PR based on the current Wasm direction. The branch remains available for reference. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
llgo run,llgo test, and comparison executables in per-invocation LLGo-owned directoriesos.Exit(0)path so build cleanup and trace defers actually run.wasmfiles after both successful and failed runner execution-oandllgo test -coutputsRoot cause
Implicit target outputs were created as isolated temp files but did not carry directory ownership, so target-generated sidecars could not be enumerated. Even after grouping them, successful runners called
mockable.Exit(0), bypassing every caller defer. That also terminated a multi-package named-target test after its first successful package.CI coverage
The single-worker wasm lane gives each public run/test invocation an isolated
TMPDIRand fails if any.mjsor.wasmexecution artifact remains. It covers:wasm, andwasip1llgo testcommand for EC32, EC64, and WC32Validation
go test ./internal/build ./cmd/internal/run ./cmd/internal/test -count=1llgo testdev/test_wasm_single_worker.shwith Binaryen wasm-opt and cache disabledbash -n dev/test_wasm_single_worker.shThis is stacked on #223 and contains no multi-worker runtime work.