dotnet/runtime

Support recognizing `Blend` as commutative

Open

#82 365 ouverte le 19 févr. 2023

Voir sur GitHub
 (5 commentaires) (2 réactions) (2 assignés)C# (5 445 forks)batch import
area-CodeGen-coreclrhelp 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

We don't currently track Blend as commutative and so we miss an opportunity to ensure that "memory operands" always appear on the RHS: https://github.com/SixLabors/ImageSharp/pull/2359#discussion_r1111147888

Support should be added to treat this as commutative which really just entails inverting the constant (e.g. Blend(x, y, 0b0111_0111) is equivalent to Blend(y, x, 0b1000_1000)).

There are actually a few operations this applies to that involve two inputs + control mask. Not all of them are as trivially as inverting the constant, however.

Guide contributeur

Direction de recherche
Étudiez l'implémentation actuelle de Blend dans le générateur de code JIT. Consultez la PR #2359 référencée et la discussion pour le contexte. Comprenez comment les opérations commutatives sont actuellement gérées et implémentez l'inversion du masque de contrôle pour les opérations compatibles. Notez que toutes les opérations avec deux entrées et un masque de contrôle ne sont pas trivialement inversibles.
Stack technique
csharp
Domaine
backend
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75