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

Commit 8f78a3b

Browse files
committed
Fix Unix preallocatedArgCount
This has been an existing ToDo for some time. This results in 19 bytes reduction in code size for pmi diffs of tests and frameworks.
1 parent 2a236b3 commit 8f78a3b

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

src/jit/morph.cpp

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4351,17 +4351,6 @@ 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-
}
43654354
#endif // UNIX_AMD64_ABI
43664355

43674356
const unsigned outgoingArgSpaceSize = preallocatedArgCount * REGSIZE_BYTES;

0 commit comments

Comments
 (0)