Skip to content

Commit

Permalink
JIT: Relax address exposure check for tailcalls (dotnet#111397)
Browse files Browse the repository at this point in the history
The `lvHasLdAddrOp` based check is conservative: address exposure (as computed in local morph) is much more precise. We can switch to that check only.
  • Loading branch information
jakobbotsch authored Jan 15, 2025
1 parent 8104cb1 commit d2cdcdd
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/coreclr/jit/morph.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5119,11 +5119,6 @@ GenTree* Compiler::fgMorphPotentialTailCall(GenTreeCall* call)
//
if (isImplicitOrStressTailCall)
{
if (varDsc->lvHasLdAddrOp && !lvaIsImplicitByRefLocal(varNum))
{
failTailCall("Local address taken", varNum);
return nullptr;
}
if (varDsc->IsAddressExposed())
{
if (lvaIsImplicitByRefLocal(varNum))
Expand Down

0 comments on commit d2cdcdd

Please sign in to comment.