Skip to content

Commit c07d793

Browse files
committed
curve25519/internal/field: fix generator module reference to x/crypto
The generator submodule needs a module dependency on golang.org/x/crypto to find the type information it needs. This removes the Comment call from CL 319469 because it does not seem to generate the intended output. See golang/go#46155. Fixes golang/go#46133 Change-Id: Iec21c6379d81271047ebf370a76329ed3fdac85c Reviewed-on: https://go-review.googlesource.com/c/crypto/+/319471 Trust: Filippo Valsorda <filippo@golang.org> Run-TryBot: Filippo Valsorda <filippo@golang.org> TryBot-Result: Go Bot <gobot@golang.org> Reviewed-by: Tobias Klauser <tobias.klauser@gmail.com>
1 parent cd7d49e commit c07d793

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

Diff for: curve25519/internal/field/_asm/fe_amd64_asm.go

+4-1
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,16 @@ import (
1111
. "github.com/mmcloughlin/avo/gotypes"
1212
. "github.com/mmcloughlin/avo/operand"
1313
. "github.com/mmcloughlin/avo/reg"
14+
15+
// Ensure "go mod tidy" doesn't remove the golang.org/x/crypto module
16+
// dependency, which is necessary to access the field.Element type.
17+
_ "golang.org/x/crypto/curve25519"
1418
)
1519

1620
//go:generate go run . -out ../fe_amd64.s -stubs ../fe_amd64.go -pkg field
1721

1822
func main() {
1923
Package("golang.org/x/crypto/curve25519/internal/field")
20-
Comment("//go:build amd64 && gc && !purego")
2124
ConstraintExpr("amd64,gc,!purego")
2225
feMul()
2326
feSquare()

Diff for: curve25519/internal/field/_asm/go.mod

+6-1
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,9 @@ module asm
22

33
go 1.16
44

5-
require github.com/mmcloughlin/avo v0.2.0
5+
require (
6+
github.com/mmcloughlin/avo v0.2.0
7+
golang.org/x/crypto v0.0.0
8+
)
9+
10+
replace golang.org/x/crypto v0.0.0 => ../../../..

Diff for: curve25519/internal/field/_asm/go.sum

+3
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,17 @@ golang.org/x/mod v0.3.0/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA=
1010
golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg=
1111
golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s=
1212
golang.org/x/net v0.0.0-20201021035429-f5854403a974/go.mod h1:sp8m0HH+o8qH0wwXwYZr8TS3Oi6o0r6Gce1SSxlDquU=
13+
golang.org/x/net v0.0.0-20210226172049-e18ecbb05110/go.mod h1:m0MpNAwzfU5UDzcl9v0D8zg8gWTRqZa9RBIspLL5mdg=
1314
golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
1415
golang.org/x/sync v0.0.0-20201020160332-67f06af15bc9/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
1516
golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
1617
golang.org/x/sys v0.0.0-20190412213103-97732733099d/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
1718
golang.org/x/sys v0.0.0-20200930185726-fdedc70b468f/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
19+
golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
1820
golang.org/x/sys v0.0.0-20210119212857-b64e53b001e4/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
1921
golang.org/x/sys v0.0.0-20210403161142-5e06dd20ab57 h1:F5Gozwx4I1xtr/sr/8CFbb57iKi3297KFs0QDbGN60A=
2022
golang.org/x/sys v0.0.0-20210403161142-5e06dd20ab57/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
23+
golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
2124
golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
2225
golang.org/x/text v0.3.3/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=
2326
golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=

0 commit comments

Comments
 (0)