[ARM64EC 5/?] Fix names of __chkstk and __security_check_cookie.

[efriedma]

Summary

Part of initial Arm64EC patchset.

Arm64EC code needs to use functions with a different name, to avoid using the x64 versions.

[DavidSpickett]

Matches https://docs.microsoft.com/en-us/windows/uwp/porting/arm64ec-abi#stack-checkers, LGTM.

[DavidSpickett]

(subject to discussion on what the final triple is but just a find/replace here)

[bcl5980]

llvm/lib/Target/AArch64/AArch64Subtarget.h

359

Do we need to use #__chkstk_arm64ec here?

[efriedma]

llvm/lib/Target/AArch64/AArch64Subtarget.h

359

Probably? I'm still not entirely sure how that interacts with the hybmp table.

[efriedma]

Rebased

[DavidSpickett]

LGTM

[bcl5980]

llvm/lib/Target/AArch64/AArch64Subtarget.h

359

main_clang.obj : error LNK2019: unresolved external symbol __chkstk_arm64ec referenced in function $iexit_thunk$cdecl$i8$varargs (EC Symbol)

It looks we do need add # here. This is the link error when I try to link a variadic function. It will disappear after I add #.

[efriedma]

llvm/lib/Target/AArch64/AArch64Subtarget.h

359

Hmm, okay... does this specifically apply to __chkstk_arm64ec and __security_check_cookie_arm64ec, or have you noticed link errors for anything else? (Trying to confirm if there's some general mangling rule we're missing, or these functions are just weird.)

[bcl5980]

llvm/lib/Target/AArch64/AArch64Subtarget.h

359

No other link errors for this case.
For now, I haven't found any other case similar to this.
But I haven't tested too many cases because my local code still has many issues.