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

Commit 1ffd468

Browse files
committed
Fix Unix preallocatedArgCount
This has been an existing ToDo for some time. This results in a .02% reduction in code size for pmi diffs of benchmarks and frameworks.
1 parent 7df363a commit 1ffd468

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
@@ -4409,17 +4409,6 @@ GenTreeCall* Compiler::fgMorphArgs(GenTreeCall* call)
44094409

44104410
#if defined(UNIX_AMD64_ABI)
44114411
opts.compNeedToAlignFrame = true; // this is currently required for the UNIX ABI to work correctly
4412-
4413-
// ToDo: Remove this re-calculation preallocatedArgCount and use the value assigned above.
4414-
4415-
// First slots go in registers only, no stack needed.
4416-
// TODO-Amd64-Unix-CQ This calculation is only accurate for integer arguments,
4417-
// and ignores floating point args (it is overly conservative in that case).
4418-
preallocatedArgCount = nonRegPassedStructSlots;
4419-
if (argSlots > MAX_REG_ARG)
4420-
{
4421-
preallocatedArgCount += argSlots - MAX_REG_ARG;
4422-
}
44234412
#endif // UNIX_AMD64_ABI
44244413

44254414
const unsigned outgoingArgSpaceSize = preallocatedArgCount * REGSIZE_BYTES;

0 commit comments

Comments
 (0)