refactor: rm JIT code & clear CI

Author: xiaost

.github/workflows/pr-check.yml

@@ -16,28 +16,6 @@ jobs:
       - name: Check Spell
         uses: crate-ci/typos@master
 
-  staticcheck:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: stable
-
-      - uses: reviewdog/action-staticcheck@v1
-        with:
-          github_token: ${{ secrets.github_token }}
-          # Change reviewdog reporter if you need [github-pr-check,github-check,github-pr-review].
-          reporter: github-pr-review
-          # Report all results.
-          filter_mode: nofilter
-          # Exit with 1 when it finds at least one finding.
-          fail_on_error: true
-          # Set staticcheck flags
-          # -ST1006 for fixing jit code receiver names like `self`
-          staticcheck_flags: -checks=inherit,-SA1029, -ST1006
-
   lint:
     runs-on: ubuntu-latest
     steps:
@@ -49,7 +27,7 @@ jobs:
 
       - name: Golangci Lint
         # https://golangci-lint.run/
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v8
         with:
           version: latest
           only-new-issues: true

.github/workflows/tests.yml

@@ -7,49 +7,14 @@ jobs:
     strategy:
       matrix:
         go: [ "1.18", oldstable, stable ]
-    runs-on: [ Linux, X64 ]
+    runs-on: [ ubuntu-latest, macos-latest ]
     steps:
       - uses: actions/checkout@v4
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.go }}
-          cache: false # don't use cache for self-hosted runners
-      - name: Test packages # only tests reflect pkg, JIT pkgs is not supported
-        run: go test -race ./internal/reflect
+      - name: Test packages
+        run: go test -race ./...
       - name: Test ./tests & Benchmark
         run: cd tests && go test -bench=. -benchmem -benchtime=100ms
-
-  unittest-arm64:
-    strategy:
-      matrix:
-        go: [ "1.18", oldstable, stable ]
-    runs-on: [ ARM64 ] # It's OK under Linux or macOS
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: ${{ matrix.go }}
-          cache: false # don't use cache for self-hosted runners
-      - name: Test packages # only tests reflect pkg, JIT pkgs is not supported
-        run: go test -race ./internal/reflect
-      - name: Test ./tests & Benchmark
-        run: cd tests && go test -bench=. -benchmem -benchtime=100ms
-
-  unittest-legacy:
-    strategy:
-      matrix:
-        go: [ "1.18", "1.23" ] # jit works <= 1.23
-    runs-on: [ Linux, X64 ]
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version: ${{ matrix.go }}
-          cache: false # don't use cache for self-hosted runners
-      - name: Build with frugal_jit
-        run: go build -v -tags=frugal_jit ./...
-      - name: Test with frugal_jit
-        run: go test -tags=frugal_jit ./...

.golangci.yml

@@ -1,3 +1 @@
-issues:
-  exclude-dirs:
-    - internal/jit
+version: "2"

debug/debug.go

@@ -38,3 +38,8 @@ type CacheStats struct {
 	Miss int
 	Size int
 }
+
+// GetStats ...
+//
+// Deprecated: It was for IT
+func GetStats() Stats { return Stats{} }

debug/debug_jit.go

@@ -25,31 +25,29 @@ import (
 
 // EncodedSize measures the encoded size of val.
 func EncodedSize(val interface{}) int {
-	if nojit {
-		return reflect.EncodedSize(val)
-	}
-	return jitEncodedSize(val)
+	return reflect.EncodedSize(val)
 }
 
 // EncodeObject serializes val into buf with Thrift Binary Protocol, with optional Zero-Copy thrift.NocopyWriter.
 // buf must be large enough to contain the entire serialization result.
 func EncodeObject(buf []byte, w thrift.NocopyWriter, val interface{}) (int, error) {
-	if nojit {
-		ret, err := reflect.Append(buf[:0], val)
-		if len(ret) > len(buf) {
-			return 0, fmt.Errorf("index out of range [%d] with length %d.\n"+
-				"Please make sure the input will not be changed after calling EncodedSize or during EncodeObject(concurrency issues).",
-				len(ret), len(buf))
-		}
-		return len(ret), err
+	ret, err := reflect.Append(buf[:0], val)
+	if len(ret) > len(buf) {
+		return 0, fmt.Errorf("index out of range [%d] with length %d.\n"+ //nolint:staticcheck // ST1005: newlines
+			"Please make sure the input will not be changed after calling EncodedSize or during EncodeObject(concurrency issues).",
+			len(ret), len(buf))
 	}
-	return jitEncodeObject(buf, w, val)
+	return len(ret), err
 }
 
 // DecodeObject deserializes buf into val with Thrift Binary Protocol.
 func DecodeObject(buf []byte, val interface{}) (int, error) {
-	if nojit {
-		return reflect.Decode(buf, val)
-	}
-	return jitDecodeObject(buf, val)
+	return reflect.Decode(buf, val)
+}
+
+// Pretouch ...
+//
+// Deprecated: It was for JIT
+func Pretouch(vt any, options ...Option) error {
+	return nil
 }

debug/debug_nojit.go

@@ -4,10 +4,8 @@ go 1.18
 
 require (
 	github.com/cloudwego/gopkg v0.1.2
-	github.com/cloudwego/iasm v0.2.0
 	github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc
 	github.com/stretchr/testify v1.9.0
-	golang.org/x/arch v0.2.0
 )
 
 require (