Skip to content

Use deterministic metadata names for emitted anonymous type fields.#49370

Merged
AlekseyTs merged 1 commit intodotnet:masterfrom
AlekseyTs:Issue48417
Nov 16, 2020
Merged

Use deterministic metadata names for emitted anonymous type fields.#49370
AlekseyTs merged 1 commit intodotnet:masterfrom
AlekseyTs:Issue48417

Conversation

@AlekseyTs
Copy link
Contributor

Fixes #48417.

@AlekseyTs AlekseyTs requested a review from a team as a code owner November 14, 2020 00:53
@jcouv jcouv added the Concept-Determinism The issue involves our ability to support determinism in binaries and PDBs created at build time. label Nov 14, 2020
Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 1)

@Youssef1313
Copy link
Member

A tracking issue to revert 9796485 may be needed.

@AlekseyTs AlekseyTs merged commit a967b49 into dotnet:master Nov 16, 2020
@ghost ghost added this to the Next milestone Nov 16, 2020
333fred added a commit to 333fred/roslyn that referenced this pull request Nov 16, 2020
* upstream/master: (148 commits)
  Remove unnecessary ArrayBuilder in MakeCallWithNoExplicitArgument (dotnet#49377)
  Revert "Skip binary for determinism checking"
  Use deterministic metadata names for emitted anonymous type fields. (dotnet#49370)
  Reuse LSP solutions if they don't need re-forking (dotnet#49305)
  Small nullability fixes (dotnet#49279)
  Create default arguments during binding (dotnet#49186)
  Clean nits
  Backport dotnet#48420 to release/dev16.8 (dotnet#49357)
  Rewrite AnalyzeFileReference.GetSupportedLanguages without LINQ (dotnet#49337)
  Use .Any extension of ImmutableArray(Of Symbol) (dotnet#48980)
  fix 'remove unnecessary cast' when doing bitwise ops on unsigned values.
  Harden, document, cross-link XunitDisposeHook
  Simplify x86 hook
  Fix split comment exporting for VB.
  Code style fix
  Overwrite xunit's app domain handling to not call AppDomain.Unload
  Revert pragma suppression
  Remove blank line
  Revert file
  Move block structure code back to Features layer
  ...
@allisonchou allisonchou modified the milestones: Next, 16.9.P2 Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Concept-Determinism The issue involves our ability to support determinism in binaries and PDBs created at build time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VB has non-deterministic output when anonymous types differ in case

5 participants