fix: small perf issue in non accellerated case with useless allocs

Consensys · Jan 14, 2025 · 987d0dd · 987d0dd
1 parent 29a9616
commit 987d0dd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 15 deletions.
diff --git a/ecc/bls12-377/fr/sis/sis.go b/ecc/bls12-377/fr/sis/sis.go
diff --git a/field/generator/internal/templates/sis/sis.go.tmpl b/field/generator/internal/templates/sis/sis.go.tmpl
@@ -113,14 +113,8 @@ func NewRSis(seed int64, logTwoDegree, logTwoBound, maxNbElementsToHash int) (*R
 				partialFFT_64[mask](k, twiddlesCoset)
 			}
 		} else {
-			cosetTable, err := r.Domain.CosetTable()
-			if err != nil {
-				return nil, err
-			}
-
-			r.smallFFT = func(k {{ .FF }}.Vector, _ uint64) {
-				k.Mul(k, {{.FF}}.Vector(cosetTable))
-				r.Domain.FFT(k, fft.DIF)
+			r.smallFFT = func(k fr.Vector, _ uint64) {
+				r.Domain.FFT(k, fft.DIF, fft.OnCoset(), fft.WithNbTasks(1))
 			}
 		}
 	{{- end}}