dotnet/roslyn

Generalize `(CollectionBuilderType<T>)[..readOnlySpan]` optimization to spread operands which are implicitly convertible to `ReadOnlySpan<T>`

Open

#73.110 aperta il 19 apr 2024

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersCode Gen QualityFeature - Collection Expressionshelp wanted

Metriche repository

Star
 (20.414 star)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

I would welcome a follow-up PR which also handles cases where the spread operand is implicitly convertible to ReadOnlySpan<T>.

Originally posted by @RikkiGibson in https://github.com/dotnet/roslyn/issues/73102#issuecomment-2067272169

Affected scenarios would include:

Span<int> span = ..;
MyCollectionBuilderType<T> collection = [..span];

// currently:
MyCollectionBuilderType<T> collection = MyCollectionBuilderType.Create((ReadOnlySpan<T>)span.ToArray());

// could change to:
MyCollectionBuilderType<T> collection = MyCollectionBuilderType.Create((ReadOnlySpan<T>)span);

Guida contributor

Direzione di ricerca
Inizia esaminando l'ottimizzazione esistente per `T[]` nella riduzione delle espressioni di raccolta del compilatore Roslyn (riferita nell'issue #73102). Individua il codice che gestisce `CollectionBuilderType` e la conversione `ReadOnlySpan`. L'obiettivo è generalizzare l'ottimizzazione a qualsiasi operando di spread implicitamente convertibile in `ReadOnlySpan<T>`, non solo agli array. Studia le condizioni attuali ed estendile in modo appropriato. Cerca i file relativi alla riduzione delle espressioni di raccolta, come `CollectionExpressionRewriter` o simili. Assicurati che la modifica mantenga la correttezza e non interrompa gli scenari esistenti.
Tech stack
csharp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50