gopls/internal/test/integration/misc: skip asm test unless a{rm,md}64
Fixes golang/go#68058
Change-Id: I8d2467474f39f1834137f5b4718cdbbca4ac0e81
Reviewed-on: https://go-review.googlesource.com/c/tools/+/593416
Reviewed-by: Michael Pratt <mpratt@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
diff --git a/gopls/internal/test/integration/misc/webserver_test.go b/gopls/internal/test/integration/misc/webserver_test.go
index c917f40..5bb709f 100644
--- a/gopls/internal/test/integration/misc/webserver_test.go
+++ b/gopls/internal/test/integration/misc/webserver_test.go
@@ -431,12 +431,16 @@
// Get the report and do some minimal checks for sensible results.
//
// Use only portable instructions below! Remember that
- // This is a test of plumbing, not compilation, so
+ // this is a test of plumbing, not compilation, so
// it's better to skip the tests, rather than refine
- // them, on any architecture that gives us trouble.
+ // them, on any architecture that gives us trouble
+ // (e.g. uses JAL for CALL, or BL<cc> for RET).
+ // We conservatively test only on the two most popular
+ // architectures.
report := get(t, doc.URI)
checkMatch(t, true, report, `TEXT.*example.com/a.f`)
- if runtime.GOARCH != "risc64" { // RISC-V uses JAL instead of CALL
+ switch runtime.GOARCH {
+ case "amd64", "arm64":
checkMatch(t, true, report, `CALL runtime.printlock`)
checkMatch(t, true, report, `CALL runtime.printstring`)
checkMatch(t, true, report, `CALL runtime.printunlock`)
