Skip to content

Commit

Permalink
[dev.regabi] test: run abi/regabipragma test with -c=1
Browse files Browse the repository at this point in the history
Currently, we call Warnl in SSA backend when we see a function
(defined or called) with regparams pragma. Calling Warnl in
concurrent environment is racy. As the debugging output is
temporary, for testing purposes we just pass -c=1. We'll remove
the pragma and the debugging print some time soon.

Change-Id: I6f925a665b953259453fc458490c5ff91f67c91a
Reviewed-on: https://go-review.googlesource.com/c/go/+/291710
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Jeremy Faller <jeremy@golang.org>
Trust: Cherry Zhang <cherryyz@google.com>
Run-TryBot: Cherry Zhang <cherryyz@google.com>
  • Loading branch information
cherrymui committed Feb 16, 2021
1 parent 03cea56 commit d3cd483
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion test/abi/regabipragma.go
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// runindir
// runindir -gcflags=-c=1
// +build !windows

// Copyright 2021 The Go Authors. All rights reserved.
Expand Down
1 change: 1 addition & 0 deletions test/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -902,6 +902,7 @@ func (t *test) run() {
if *linkshared {
cmd = append(cmd, "-linkshared")
}
cmd = append(cmd, flags...)
cmd = append(cmd, ".")
out, err := runcmd(cmd...)
if err != nil {
Expand Down

0 comments on commit d3cd483

Please sign in to comment.