Skip to content

Commit

Permalink
Go back to typing the "first field map" as a field
Browse files Browse the repository at this point in the history
This is a temporary change to minimize diffs.
The issue is that we need to update both the
reading and writing code at once if we want
to change the types of the maps without some
regressions due to new artificial mismatches.
  • Loading branch information
SingleAccretion committed Nov 6, 2021
1 parent 3744a29 commit 352acf4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/coreclr/jit/valuenum.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7795,8 +7795,8 @@ void Compiler::fgValueNumberAssignment(GenTreeOp* tree)

// Construct the "field map" VN. It represents memory state of the first field
// of all objects on the heap. This is our primary map.
ValueNum fldMapVN = vnStore->VNForMapSelect(VNK_Liberal, TYP_REF, fgCurMemoryVN[GcHeap],
firstFieldSelectorVN);
ValueNum fldMapVN = vnStore->VNForMapSelect(VNK_Liberal, firstFieldType,
fgCurMemoryVN[GcHeap], firstFieldSelectorVN);

ValueNum firstFieldValueSelectorVN = ValueNumStore::NoVN;
if (obj != nullptr)
Expand Down

0 comments on commit 352acf4

Please sign in to comment.