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.

Sign up for GitHub

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

Jump to bottom

Assertion argCnt < MAX_PTRARG_OFS failed #112251

Open
MichalStrehovsky opened this issue Feb 6, 2025 · 2 comments · May be fixed by #112268
Open

Assertion argCnt < MAX_PTRARG_OFS failed #112251

MichalStrehovsky opened this issue Feb 6, 2025 · 2 comments · May be fixed by #112268
Assignees
@filipnavara
Labels
arch-x86 area-NativeAOT-coreclr disabled-test The test is disabled in source code against the issue in-pr There is an active PR which will close this issue when it is merged os-windows untriaged New issue has not been triaged by the area owner

Comments

@MichalStrehovsky
Copy link
Member

Failing in the x86 pri1 test runs. This looks like regular C# code, not clear what the problem is.

The test in question is https://github.com/dotnet/runtime/tree/9c2bfb9ab73961d149ea263c7a78606bfb4bc0b4/src/tests/JIT/Regression/CLR-x86-JIT/v2.1/DDB/b202743.

---------------------------
Microsoft Visual C++ Runtime Library
---------------------------
Debug Assertion Failed!

Program: ...-2a44d98b000a\Regression_1\workitems\Regression_1\b202743.exe
File: D:\a\_work\1\s\src\coreclr\vm\gc_unwind_x86.inl
Line: 1826

Expression: argCnt < MAX_PTRARG_OFS

For information on how your program can cause an assertion
failure, see the Visual C++ documentation on asserts.

(Press Retry to debug the application)

---------------------------
Abort   Retry   Ignore   
---------------------------
@MichalStrehovsky MichalStrehovsky added arch-x86 area-NativeAOT-coreclr disabled-test The test is disabled in source code against the issue os-windows labels Feb 6, 2025
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Feb 6, 2025
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

@filipnavara
Copy link
Member

filipnavara commented Feb 7, 2025 •
edited
Loading

I never ported the bitvector code used for large number of method arguments. Looks like it finally caught me up. Weirdly the test just uses one large struct argument but I suppose it's some ABI thing I overlooked.

@filipnavara filipnavara self-assigned this Feb 7, 2025
@filipnavara filipnavara linked a pull request Feb 7, 2025 that will close this issue
Open
@dotnet-policy-service dotnet-policy-service bot added the in-pr There is an active PR which will close this issue when it is merged label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@filipnavara filipnavara

Labels
arch-x86 area-NativeAOT-coreclr disabled-test The test is disabled in source code against the issue in-pr There is an active PR which will close this issue when it is merged os-windows untriaged New issue has not been triaged by the area owner
Projects
AppModel
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace bitvector.h/cpp with ptrArgTP type in gc_unwind_x86.h/inl filipnavara/runtime
2 participants
@filipnavara @MichalStrehovsky