dotnet / dotnet/machinelearning
CpuMath Enhancement: Remove any unnecessary Contracts.Assert covered by Span<T> in hardware intrinsics APIs
Open
area-Native
Priority:3
- Dominant language
- C#
- Stars
- 9.4k
- Forks
- 2k
- Avg merge
- 2d 20h
- Merged PRs (30d)
- 11
Description
Style changes needed to solve part of https://github.com/dotnet/machinelearning/issues/823
## Details
- In `src\Microsoft.ML.CpuMath\CpuMathUtils.netcoreapp.cs`, `Span` might already do the checks (`Contracts.Assert`) in the public functions. Reference:
https://github.com/dotnet/machinelearning/blob/e443e2afcd39a8a363321d423e087c0569c0d4af/src/Microsoft.ML.CpuMath/CpuMathUtils.netcoreapp.cs#L237-L245
Contributor guide
Assessment
This issue has not been assessed yet.