Skip to content

[release/dev18.3] Use command line checksumAlgorithm in generator driver#81954

Merged
RikkiGibson merged 8 commits intorelease/dev18.3from
backport/pr-81934-to-release/dev18.3
Jan 12, 2026
Merged

[release/dev18.3] Use command line checksumAlgorithm in generator driver#81954
RikkiGibson merged 8 commits intorelease/dev18.3from
backport/pr-81934-to-release/dev18.3

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 10, 2026

Backport of #81934 to release/dev18.3

/cc @RikkiGibson

Customer Impact

Today users are finding that source generated files can use a different checksum algorithm in PDBs than ordinary files use in the same compilation. Prior to this change, there's no way for source generator users to control which checksum algorithm is used for the generated files. This is problematic because certain builds need to restrict which checksum algorithms are permitted to be used or not.

This change makes it so the same checksum algorithm determined by compiler options will be used for all files in the compilation, both ordinary and source-generated.

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

Automated tests are included for the scenario. The issue was already known for a few years, but not thought to be blocking until recently.

Risk

Low. We consider it unlikely that components downstream of the generated files are relying on a generator-determined checksum algorithm, instead of one determined by compiler options.

@github-actions github-actions bot requested a review from a team as a code owner January 10, 2026 03:08
@RikkiGibson RikkiGibson merged commit 1818262 into release/dev18.3 Jan 12, 2026
25 checks passed
@RikkiGibson RikkiGibson deleted the backport/pr-81934-to-release/dev18.3 branch January 12, 2026 23:45
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