golang/go

crypto/cipher: optimize safeXORBytes

Aperta

#35.381 aperta il 5 nov 2019

Vedi su GitHubGuida contributor
 (11 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
NeedsInvestigationPerformanceSuggestedhelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The discussion at https://github.com/golang/go/issues/31586 includes several techniques for optimizing xor of byte slices, including using encoding/binary (with native endianness) and loop unrolling. It'd be nice to apply them to safeXORBytes, which is used on many non-amd64 platforms. An optimized safeXORBytes might even be fast enough that we could delete fastXORBytes.

Somewhat related: #30553

cc @nhooyr

Guida contributor

Direzione di ricerca
Esamina l'implementazione di safeXORBytes in crypto/cipher/. Rivedi le tecniche di ottimizzazione discusse in #31586, in particolare l'uso di encoding/binary con endianness nativa e lo srotolamento dei cicli. Esegui benchmark della funzione attuale e delle modifiche proposte. Valuta se fastXORBytes può essere rimosso se safeXORBytes è abbastanza veloce.
Tech stack
go
Dominio
backend
Tipo issue
Performance
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
Gocrypto/cipher
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40