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 failed 'sizeBytes != 0' #72081

Closed
MichalStrehovsky opened this issue Jul 13, 2022 · 1 comment · Fixed by #72084
Closed

Assertion failed 'sizeBytes != 0' #72081

MichalStrehovsky opened this issue Jul 13, 2022 · 1 comment · Fixed by #72084
Assignees
@EgorBo
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Milestone
7.0.0

Comments

@MichalStrehovsky
Copy link
Member

Ran into this while NativeAOT compiling the System.Runtime.Intrinsics.Tests but this has a simple repro with CoreCLR. Not clear why we're not seeing this in test runs.

Compile following:

using System;
using System.Runtime.Intrinsics;

Vector128<UInt128> d = default;
d.GetHashCode();

With csc test.cs /noconfig /nostdlib /r:System.Private.CoreLib.dll /O. Run with corerun.

Assert failure(PID 17384 [0x000043e8], Thread: 40284 [0x9d5c]): Assertion failed 'sizeBytes != 0' in 'System.Runtime.Intrinsics.Vector128`1[UInt128][System.UInt128]:GetHashCode():int:this' during 'Importation' (IL size 67; hash 0x04fdae41; MinOpts)

    File: C:\git\runtime2\src\coreclr\jit\hwintrinsic.cpp Line: 777
    Image: C:\git\runtime2\artifacts\bin\coreclr\windows.x64.Debug\corerun.exe
@MichalStrehovsky MichalStrehovsky added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Jul 13, 2022
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jul 13, 2022
@ghost
Copy link

ghost commented Jul 13, 2022

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

Issue Details

Ran into this while NativeAOT compiling the System.Runtime.Intrinsics.Tests but this has a simple repro with CoreCLR. Not clear why we're not seeing this in test runs.

Compile following:

using System;
using System.Runtime.Intrinsics;

Vector128<UInt128> d = default;
d.GetHashCode();

With csc test.cs /noconfig /nostdlib /r:System.Private.CoreLib.dll /O. Run with corerun.

Assert failure(PID 17384 [0x000043e8], Thread: 40284 [0x9d5c]): Assertion failed 'sizeBytes != 0' in 'System.Runtime.Intrinsics.Vector128`1[UInt128][System.UInt128]:GetHashCode():int:this' during 'Importation' (IL size 67; hash 0x04fdae41; MinOpts)

    File: C:\git\runtime2\src\coreclr\jit\hwintrinsic.cpp Line: 777
    Image: C:\git\runtime2\artifacts\bin\coreclr\windows.x64.Debug\corerun.exe
Author: MichalStrehovsky
Assignees: -
Labels:

area-CodeGen-coreclr
Milestone: -

@EgorBo EgorBo mentioned this issue Jul 13, 2022
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 13, 2022
@MichalStrehovsky MichalStrehovsky mentioned this issue Jul 14, 2022
Closed
@EgorBo EgorBo added this to the 7.0.0 milestone Jul 14, 2022
@EgorBo EgorBo removed the untriaged New issue has not been triaged by the area owner label Jul 14, 2022
@EgorBo EgorBo self-assigned this Jul 14, 2022
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 14, 2022
MichalStrehovsky added a commit to MichalStrehovsky/runtime that referenced this issue Jul 27, 2022
@MichalStrehovsky
Get more libraries tests running with NativeAOT
4f6abec 
* MetadataLoadContext tests need more work for single file testing in general. Filed a separate issue.
* S.R.Intrinsics is now clean. Running it on NativeAOT found good JIT bugs (dotnet#72081, dotnet#72158). Promoting to a smoke test.
* S.Drawing.Primitives was testing DataContractSerializer for whatever reason. Marked that as aggresive trimming incompatible. This wouldn't be covered by dotnet#72910.
* Made S.R.InteropServices.ComDisabled fully pass by fixing exceptions we throw.
* A couple tests won't ever make sense to run on NativeAOT so removed them from triage area.
@MichalStrehovsky MichalStrehovsky mentioned this issue Jul 27, 2022
Merged
MichalStrehovsky added a commit that referenced this issue Jul 29, 2022
@MichalStrehovsky
Get more libraries tests running with NativeAOT (#72917)
7d1e1af 
* MetadataLoadContext tests need more work for single file testing in general. Filed a separate issue.
* S.R.Intrinsics is now clean. Running it on NativeAOT found good JIT bugs (#72081, #72158). Promoting to a smoke test.
* S.Drawing.Primitives was testing DataContractSerializer for whatever reason. Marked that as aggresive trimming incompatible. This wouldn't be covered by #72910.
* Made S.R.InteropServices.ComDisabled fully pass by fixing exceptions we throw.
* A couple tests won't ever make sense to run on NativeAOT so removed them from triage area.
@ghost ghost locked as resolved and limited conversation to collaborators Aug 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@EgorBo EgorBo

Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

Remove an assert from impHWIntrinsic EgorBo/runtime-1
2 participants
@EgorBo @MichalStrehovsky