dotnet/runtime

Improve APIs for Intel string handling intrinsics

Open

#957 ouverte le 13 juin 2018

Voir sur GitHub
 (14 commentaires) (2 réactions) (0 assignés)C# (5 445 forks)batch import
arch-x64area-System.Runtime.Intrinsicshelp wanted

Métriques du dépôt

Stars
 (17 886 stars)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

Description

See https://github.com/dotnet/coreclr/pull/17637 for some discussion and context. The fundamental issues revolve around how much of the behavior should be controlled by enums passed into the intrinsic method, versus separating the different behaviors into differently-named intrinsics.

The current API includes both StringComparisonMode (https://github.com/dotnet/corefx/blob/master/src/System.Runtime.Intrinsics/ref/System.Runtime.Intrinsics.cs#L864) and ResultsFlag (https://github.com/dotnet/corefx/blob/master/src/System.Runtime.Intrinsics/ref/System.Runtime.Intrinsics.cs#L875).

Some of the open questions are:

  • Should the ResultsFlag, which determines the comparison results for which a true result is returned, become part of the method name?
  • How should these be named? According to the instruction documentation (e.g. CFlag, as in the current API), or something more descriptive like CompareNonZeroMask?
  • Should these be combined into a single enum or remain separate?

Guide contributeur

Direction de recherche
Examinez la PR liée (dotnet/coreclr#17637) pour le contexte, examinez la conception actuelle de l'API pour les intrinsèques de gestion des chaînes Intel, y compris StringComparisonMode et ResultsFlag, et proposez des solutions aux questions ouvertes concernant la dénomination et la combinaison des énumérations.
Stack technique
csharp
Domaine
backend api design
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#.NET runtime intrinsics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20