Skip to content

Add wasm_coreclr and coreclr_r2r_interpreter to perf-build pipeline#126589

Open
LoopedBard3 wants to merge 1 commit intodotnet:mainfrom
LoopedBard3:UpdatePerfBuildsToIncludeWasmCoreclr
Open

Add wasm_coreclr and coreclr_r2r_interpreter to perf-build pipeline#126589
LoopedBard3 wants to merge 1 commit intodotnet:mainfrom
LoopedBard3:UpdatePerfBuildsToIncludeWasmCoreclr

Conversation

@LoopedBard3
Copy link
Copy Markdown
Member

Include the two new build types (wasm_coreclr and coreclr_r2r_interpreter) in the every-commit perf build pipeline so they are registered, built, and uploaded to the build cache system (BCS).

Should be merged with: dotnet/performance#5191
Test run: https://dev.azure.com/dnceng/internal/_build/results?buildId=2944775&view=results

Include the two new build types (wasm_coreclr and coreclr_r2r_interpreter)
in the every-commit perf build pipeline so they are registered, built,
and uploaded to the build cache system (BCS).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @agocke, @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the every-commit perf build pipeline to include two additional runtime build types so they get registered, built, and uploaded to the Build Cache System (BCS).

Changes:

  • Adds new pipeline parameters to enable/disable wasm_coreclr and coreclr_r2r_interpreter builds.
  • Threads these build types through the RegisterBuild, Build, and UploadArtifacts stage template invocations.

Comment on lines +151 to +152
- ${{ if eq(parameters.coreclr_r2r_interpreter, true) }}:
- coreclr_r2r_interpreter
Copy link

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

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

These new buildType values are passed into templates from the performance repo (e.g., /eng/pipelines/runtime-perf-build-jobs.yml@performance), but this pipeline doesn’t pin resources.repositories.performance.ref. If this PR merges before the corresponding dotnet/performance changes, the pipeline can break due to unknown/unhandled build types. Consider pinning ref to a commit/branch that contains the required template updates until the performance-side PR is merged, or otherwise ensure merge order prevents a broken main build.

Suggested change
- ${{ if eq(parameters.coreclr_r2r_interpreter, true) }}:
- coreclr_r2r_interpreter

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants