Skip to content

Conversation

@am11
Copy link
Member

@am11 am11 commented Mar 4, 2024

Without this patch, cross-compilation for linux-riscv64 from linux-arm64 fails like this:

$ uname -m
aarch64

$ ROOTFS_DIR=/crossrootfs/riscv64 src/coreclr/build-runtime.sh --cross --arch riscv64

[ 11%] Building CXX object md/runtime/CMakeFiles/mdruntime_wks.dir/recordpool.cpp.o
In file included from /runtime/src/coreclr/dlls/mscorrc/include.rc:4:
In file included from /runtime/src/coreclr/dlls/mscorrc/mscorrc.rc:7:
In file included from /runtime/src/coreclr/pal/prebuilt/inc/corerror.h:7:
In file included from /runtime/src/coreclr/pal/inc/rt/winerror.h:12:
In file included from /runtime/src/coreclr/pal/inc/rt/palrt.h:136:
/runtime/src/coreclr/pal/inc/pal.h:2764:2: error: PAL_CS_NATIVE_DATA_SIZE is not defined for this architecture
 2764 | #error  PAL_CS_NATIVE_DATA_SIZE is not defined for this architecture
      |  ^
1 error generated.

and with the patch, coreclr build succeeds.

@ghost
Copy link

ghost commented Mar 4, 2024

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

Without this patch, cross-compilation for linux-riscv64 from linux-arm64 fails like this:

$ uname -m
aarch64

$ ROOTFS_DIR=/crossrootfs/riscv64 src/coreclr/build-runtime.sh --cross --arch riscv64

[ 11%] Building CXX object md/runtime/CMakeFiles/mdruntime_wks.dir/recordpool.cpp.o
In file included from /runtime/src/coreclr/dlls/mscorrc/include.rc:4:
In file included from /runtime/src/coreclr/dlls/mscorrc/mscorrc.rc:7:
In file included from /runtime/src/coreclr/pal/prebuilt/inc/corerror.h:7:
In file included from /runtime/src/coreclr/pal/inc/rt/winerror.h:12:
In file included from /runtime/src/coreclr/pal/inc/rt/palrt.h:136:
/runtime/src/coreclr/pal/inc/pal.h:2764:2: error: PAL_CS_NATIVE_DATA_SIZE is not defined for this architecture
 2764 | #error  PAL_CS_NATIVE_DATA_SIZE is not defined for this architecture
      |  ^
1 error generated.
Author: am11
Assignees: -
Labels:

area-Infrastructure-coreclr

Milestone: -

Copy link
Member

@jkoritzinsky jkoritzinsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jkoritzinsky jkoritzinsky merged commit 77c22ee into dotnet:main Mar 6, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Apr 7, 2024
@am11 am11 deleted the patch-28 branch June 14, 2024 04:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants