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

[x86/Linux] Declare GetCallerSp when WIN64EXCEPTION (not _TARGET_X86_) is defined#8372

Closed
parjong wants to merge 1 commit into
dotnet:masterfrom
parjong:fix/eetwain_header
Closed

[x86/Linux] Declare GetCallerSp when WIN64EXCEPTION (not _TARGET_X86_) is defined#8372
parjong wants to merge 1 commit into
dotnet:masterfrom
parjong:fix/eetwain_header

Conversation

@parjong
Copy link
Copy Markdown

@parjong parjong commented Nov 30, 2016

GetExactGenericsToken in eetwain.cpp (enclosed by WIN64EXCEPTION) calls
GetCallerSp, but eetwain.h declares GetCallerSp only when TARGET_X86
is defined.

This results in compile error for x86/Linux build.

This commit revises eetwain.h to declare GetCallerSp when WIN64EXCEPTION
is defined.

GetExactGenericsToken in eetwain.cpp (enclosed by WIN64EXCEPTION) calls
GetCallerSp, but eetwain.h declares GetCallerSp only when _TARGET_X86_
is defined.

This results in compile error for x86/Linux build.

This commit revises eetwain.h to declare GetCallerSp when WIN64EXCEPTION
is defined.
@parjong
Copy link
Copy Markdown
Author

parjong commented Nov 30, 2016

\CC @seanshpark

@parjong parjong changed the title [x86/Linux] Declares GetCallerSp for WIN64EXCEPTION (not for _TARGET_X86_) [x86/Linux] Declare GetCallerSp when WIN64EXCEPTION (not _TARGET_X86_) is defined Nov 30, 2016
@jkotas
Copy link
Copy Markdown
Member

jkotas commented Dec 1, 2016

This should go together with a matching fix in the .cpp file.

Let's close this for now and focus on the !WIN64EXCEPTIONS path.

@jkotas jkotas closed this Dec 1, 2016
@parjong parjong deleted the fix/eetwain_header branch December 14, 2016 07:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants