Merge pull request #111 from nvanbenschoten/nvanbenschoten/gcassert

apd: use gcassert to check that tmp big.Ints don't escape, add to CI
cockroachdb · Jan 10, 2022 · a3aa8bd · a3aa8bd
2 parents 17328c1 + c93f42c
commit a3aa8bd
Show file tree

Hide file tree

Showing 2 changed files with 71 additions and 57 deletions.
diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml
@@ -65,6 +65,14 @@ jobs:
           go get honnef.co/go/tools/cmd/staticcheck
           staticcheck ./...
 
+      - name: 'GCAssert'
+        # Only run gcassert on the latest versions of Go. Inlining heuristics
+        # change from version to version.
+        if: ${{ matrix.arch == 'x64' && matrix.go >= '1.17' }}
+        run: |
+          go get github.com/jordanlewis/gcassert/cmd/gcassert
+          gcassert ./...
+
       - name: 'BuildTest for armv7'
         if: ${{ matrix.arch == 'armv7' }}
         env: