VSadov/Satori
1
0
Fork 0
mirror of https://github.com/VSadov/Satori.git synced 2025-06-11 18:20:26 +09:00
Code Activity

fix condition to align method at 32 bytes (#42909)

Browse source 
In #2249, we started doing alignment of methods to 32-byte boundary for Tier1. However, a method having loops bypass tiering and hence this condition was never executed. Fixed it to make sure we do the alignment for optimized methods only and don't do it for prejitted methods.
This commit is contained in:
Kunal Pathak 2020-10-07 11:02:14 -07:00 • committed by GitHub
parent 65a27aa018
commit 03a09319b7
Signed by: github
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/coreclr/src/jit/emit.cpp
View file

@ -4771,10 +4771,11 @@ unsigned emitter::emitEndCodeGen(Compiler* comp,
#endif
#ifdef TARGET_XARCH
// For x64/x86, align Tier1 methods to 32 byte boundaries if
// For x64/x86, align methods that are "optimizations enabled" to 32 byte boundaries if
// they are larger than 16 bytes and contain a loop.
//
if (emitComp->opts.jitFlags->IsSet(JitFlags::JIT_FLAG_TIER1) && (emitTotalHotCodeSize > 16) && emitComp->fgHasLoops)
if (emitComp->opts.OptimizationEnabled() && !emitComp->opts.jitFlags->IsSet(JitFlags::JIT_FLAG_PREJIT) &&
(emitTotalHotCodeSize > 16) && emitComp->fgHasLoops)
{
allocMemFlag = CORJIT_ALLOCMEM_FLG_32BYTE_ALIGN;
}