dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
14.85k stars 4.62k forks source link

JIT: SVE Cleanup - Remove unneeded comments from API surface area #104846

Closed TIHan closed 4 weeks ago

TIHan commented 1 month ago

In these files:

There are comments related to the SVE APIs that are not part of the summary blocks. Example:

        ///  CreateTrueMaskDouble : Set predicate elements to true

        /// <summary>
        /// svbool_t svptrue_pat_b8(enum svpattern pattern)
        ///   PTRUE Presult.B, pattern
        /// </summary>
        public static unsafe Vector<double> CreateTrueMaskDouble([ConstantExpected] SveMaskPattern pattern = SveMaskPattern.All) => CreateTrueMaskDouble(pattern);

We need to remove the /// CreateTrueMaskDouble : Set predicate elements to true line.

dotnet-policy-service[bot] commented 1 month ago

Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics See info in area-owners.md if you want to be subscribed.

a74nh commented 4 weeks ago

We need to remove the /// CreateTrueMaskDouble : Set predicate elements to true line.

We just change it to a double quote // CreateTrueMaskDouble : Set predicate elements to true ??

a74nh commented 4 weeks ago

priority:2 for RC1 snap : End users will see the API