Skip to content

[cmake][interop][win] Fix compiler error C1128#22833

Merged
bellenot merged 1 commit into
root-project:masterfrom
bellenot:fix-cppinterop-win
Jul 17, 2026
Merged

[cmake][interop][win] Fix compiler error C1128#22833
bellenot merged 1 commit into
root-project:masterfrom
bellenot:fix-cppinterop-win

Conversation

@bellenot

Copy link
Copy Markdown
Member

Fix the following error on Windows when compiling with ASAN=ON:

interpreter\CppInterOp\lib\CppInterOp\CppInterOp.cpp(1,1): error C1128:
number of sections exceeded object file format limit: compile with /bigobj

Fix the following error on Windows when compiling with ASAN=ON:
interpreter\CppInterOp\lib\CppInterOp\CppInterOp.cpp(1,1): error C1128:
number of sections exceeded object file format limit: compile with /bigobj
@bellenot
bellenot requested a review from aaronj0 July 16, 2026 11:19
@bellenot bellenot self-assigned this Jul 16, 2026
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 11h 0m 30s ⏱️
 3 872 tests  3 872 ✅ 0 💤 0 ❌
76 180 runs  76 180 ✅ 0 💤 0 ❌

Results for commit 152ad88.

@aaronj0 aaronj0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@bellenot

Copy link
Copy Markdown
Member Author

@aaronj0 thanks! But what why is Diff CppInterOp against upstream / cppinterop-diff (pull_request) not happy?

@aaronj0

aaronj0 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@aaronj0 thanks! But what why is Diff CppInterOp against upstream / cppinterop-diff (pull_request) not happy?

We can merge, this will be included in the next upgrade

@bellenot
bellenot merged commit 739a704 into root-project:master Jul 17, 2026
35 of 36 checks passed
@bellenot
bellenot deleted the fix-cppinterop-win branch July 17, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants