Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit efd7220

Browse files
authored
Merge pull request #19276 from CarolEidt/FixUxArgSize
Fix Unix preallocatedArgCount
2 parents 4a5e35b + 4e26024 commit efd7220

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

src/jit/morph.cpp

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4351,18 +4351,7 @@ GenTreeCall* Compiler::fgMorphArgs(GenTreeCall* call)
43514351

43524352
#if defined(UNIX_AMD64_ABI)
43534353
opts.compNeedToAlignFrame = true; // this is currently required for the UNIX ABI to work correctly
4354-
4355-
// ToDo: Remove this re-calculation preallocatedArgCount and use the value assigned above.
4356-
4357-
// First slots go in registers only, no stack needed.
4358-
// TODO-Amd64-Unix-CQ This calculation is only accurate for integer arguments,
4359-
// and ignores floating point args (it is overly conservative in that case).
4360-
preallocatedArgCount = nonRegPassedStructSlots;
4361-
if (argSlots > MAX_REG_ARG)
4362-
{
4363-
preallocatedArgCount += argSlots - MAX_REG_ARG;
4364-
}
4365-
#endif // UNIX_AMD64_ABI
4354+
#endif // UNIX_AMD64_ABI
43664355

43674356
const unsigned outgoingArgSpaceSize = preallocatedArgCount * REGSIZE_BYTES;
43684357
call->fgArgInfo->SetOutArgSize(max(outgoingArgSpaceSize, MIN_ARG_AREA_FOR_CALL));

0 commit comments

Comments
 (0)