dotnet/runtime

Support recognizing `Blend` as commutative

Open

#82.365 aperta il 19 feb 2023

Vedi su GitHub
 (5 commenti) (2 reazioni) (2 assegnatari)C# (5445 fork)batch import
area-CodeGen-coreclrhelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga l'implementazione attuale di Blend nel generatore di codice JIT. Esamina il PR #2359 citato e la relativa discussione per il contesto. Comprendi come vengono attualmente gestite le operazioni commutative e implementa l'inversione della maschera di controllo per le operazioni compatibili. Nota che non tutte le operazioni con due ingressi e una maschera di controllo sono banalmente invertibili.
Tech stack
csharp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75