Skip to content

Commit

Permalink
Fix failed assertion 'FPbased == FPbased2' (#111787)
Browse files Browse the repository at this point in the history
* Move assert to prevent incorrect trigger when lvaFrameAddress optimization merged SP+n and FP-m stores into a single pair-wise store

* Better fix
  • Loading branch information
filipnavara authored Jan 30, 2025
1 parent f3d90b2 commit 7693b6d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/coreclr/jit/emitarm64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -12311,7 +12311,7 @@ size_t emitter::emitOutputInstr(insGroup* ig, instrDesc* id, BYTE** dp)

// If there are 2 GC vars in this instrDesc, get the 2nd variable
// that should be tracked.
adr2 = emitComp->lvaFrameAddress(varNum2, &FPbased2, true);
adr2 = emitComp->lvaFrameAddress(varNum2, &FPbased2, FPbased);
ofs2Dist = EA_SIZE_IN_BYTES(size);
#ifdef DEBUG
assert(FPbased == FPbased2);
Expand Down

0 comments on commit 7693b6d

Please sign in to comment.