Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crossgen method bodies having ARM64 intrinsic #38060

Merged
merged 6 commits into from
Jun 20, 2020
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions src/coreclr/src/pal/src/misc/jitsupport.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,15 @@ PAL_GetJitCpuCapabilityFlags(CORJIT_FLAGS *flags)
_ASSERTE(flags);

CORJIT_FLAGS &CPUCompileFlags = *flags;


#if defined(HOST_ARM64)

// Enable ARM64 based flags by default so we always crossgen
// ARM64 intrinsics for Linux
CPUCompileFlags.Set(InstructionSet_ArmBase);
CPUCompileFlags.Set(InstructionSet_AdvSimd);

kunalspathak marked this conversation as resolved.
Show resolved Hide resolved
#if HAVE_AUXV_HWCAP_H
unsigned long hwCap = getauxval(AT_HWCAP);

Expand All @@ -32,7 +40,6 @@ PAL_GetJitCpuCapabilityFlags(CORJIT_FLAGS *flags)
// From a single binary distribution perspective, compiling with latest kernel asm/hwcap.h should
// include all published flags. Given flags are merged to kernel and published before silicon is
// available, using the latest kernel for release should be sufficient.
CPUCompileFlags.Set(InstructionSet_ArmBase);
#ifdef HWCAP_AES
if (hwCap & HWCAP_AES)
CPUCompileFlags.Set(InstructionSet_Aes);
Expand Down Expand Up @@ -117,13 +124,6 @@ PAL_GetJitCpuCapabilityFlags(CORJIT_FLAGS *flags)
// if (hwCap & HWCAP_SVE)
// CPUCompileFlags.Set(CORJIT_FLAGS::CORJIT_FLAG_HAS_ARM64_SVE);
#endif
#else // !HAVE_AUXV_HWCAP_H
// CoreCLR SIMD and FP support is included in ARM64 baseline
// On exceptional basis platforms may leave out support, but CoreCLR does not
// yet support such platforms
// Set baseline flags if OS has not exposed mechanism for us to determine CPU capabilities
CPUCompileFlags.Set(InstructionSet_AdvSimd);
// CPUCompileFlags.Set(CORJIT_FLAGS::CORJIT_FLAG_HAS_ARM64_FP);
#endif // HAVE_AUXV_HWCAP_H
#endif // defined(HOST_ARM64)
CPUCompileFlags.Set64BitInstructionSetVariants();
Expand Down
4 changes: 2 additions & 2 deletions src/coreclr/src/vm/methodtablebuilder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1481,10 +1481,10 @@ MethodTableBuilder::BuildMethodTableThrowing(
#endif
{
#if defined(CROSSGEN_COMPILE)
#if defined(TARGET_X86) || defined(TARGET_AMD64)
#if defined(TARGET_X86) || defined(TARGET_AMD64) || defined(TARGET_ARM64)
if ((!IsNgenPDBCompilationProcess()
&& GetAppDomain()->ToCompilationDomain()->GetTargetModule() != g_pObjectClass->GetModule()))
#endif // defined(TARGET_X86) || defined(TARGET_AMD64)
#endif // defined(TARGET_X86) || defined(TARGET_AMD64) || defined(TARGET_ARM64)
{
// Disable AOT compiling for managed implementation of hardware intrinsics.
// We specially treat them here to ensure correct ISA features are set during compilation
Expand Down
11 changes: 9 additions & 2 deletions src/coreclr/src/zap/zapinfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2147,10 +2147,17 @@ DWORD FilterNamedIntrinsicMethodAttribs(ZapInfo* pZapInfo, DWORD attribs, CORINF
// is because they often change the code they emit based on what ISAs are supported by the compiler,
// but we don't know what the target machine will support.
//
// Additionally, we make sure none of the hardware intrinsic method bodies get pregenerated in crossgen
// Additionally, we make sure none of the hardware intrinsic method bodies (except ARM64) get pregenerated in crossgen
// (see ZapInfo::CompileMethod) but get JITted instead. The JITted method will have the correct
// answer for the CPU the code is running on.
fTreatAsRegularMethodCall = (fIsGetIsSupportedMethod && fIsPlatformHWIntrinsic) || (!fIsPlatformHWIntrinsic && fIsHWIntrinsic);

// For Arm64, AdvSimd/ArmBase is the baseline that is suported and hence we do pregenerate the method bodies
// of ARM4 harware intrinsic.
fTreatAsRegularMethodCall = (fIsGetIsSupportedMethod && fIsPlatformHWIntrinsic);
#if !defined(TARGET_ARM64)
fTreatAsRegularMethodCall |= (!fIsPlatformHWIntrinsic && fIsHWIntrinsic);
kunalspathak marked this conversation as resolved.
Show resolved Hide resolved
#endif


if (fIsPlatformHWIntrinsic)
{
Expand Down