panic on Go1.15

bytedance · Apr 4, 2023 · 4e42777 · 4e42777
1 parent 6ce8eeb
commit 4e42777
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 165 deletions.
diff --git a/internal/native/dispatch_amd64.go b/internal/native/dispatch_amd64.go
@@ -1,6 +1,3 @@
-//go:build go1.16 && amd64
-// +build go1.16,amd64
-
 /*
  * Copyright 2021 ByteDance Inc.
  *

diff --git a/internal/native/dispatch_compat.go b/internal/native/dispatch_compat.go
diff --git a/loader/wrapper_c.go b/loader/wrapper_c.go
@@ -26,14 +26,25 @@ import (
 
 var _C_Redzone = []bool{false, false, false, false}
 
+// CFunc is a function information for C func
 type CFunc struct {
+    // C function name
     Name     string
+
+    // entry pc relative to entire text segment
     EntryOff uint32
+
+    // function text size in bytes
     TextSize uint32
+
+    // maximum stack depth of the function
     MaxStack uintptr
+
+    // PC->SP delta lists of the function
     Pcsp     [][2]uint32
 }
 
+// GoC is the wrapper for Go calls to C
 type GoC struct {
     // CName is the name of corresponding C function
     CName     string
@@ -51,6 +62,7 @@ type GoC struct {
     GoFunc   interface{} 
 }
 
+// WrapGoC wraps C functions and loader it into Go stubs
 func WrapGoC(text []byte, natives []CFunc, stubs []GoC, modulename string, filename string) {
     funcs := make([]Func, len(natives))