From 5282b4efac70466bf372676ae2f99fa065978490 Mon Sep 17 00:00:00 2001
From: Ayke van Laethem <aykevanlaethem@gmail.com>
Date: Mon, 17 Mar 2025 14:26:48 +0100
Subject: [PATCH] runtime: remove unused file func.go

This was used in the past, but we don't use it anymore.
---
 compiler/llvm.go    |  7 -------
 interp/memory.go    | 13 -------------
 src/runtime/func.go | 28 ----------------------------
 3 files changed, 48 deletions(-)
 delete mode 100644 src/runtime/func.go

diff --git a/compiler/llvm.go b/compiler/llvm.go
index 59aaee8fd..139c5a1cd 100644
--- a/compiler/llvm.go
+++ b/compiler/llvm.go
@@ -371,13 +371,6 @@ func (c *compilerContext) getPointerBitmap(typ llvm.Type, pos token.Pos) *big.In
 		return big.NewInt(1)
 	case llvm.StructTypeKind:
 		ptrs := big.NewInt(0)
-		if typ.StructName() == "runtime.funcValue" {
-			// Hack: the type runtime.funcValue contains an 'id' field which is
-			// of type uintptr, but before the LowerFuncValues pass it actually
-			// contains a pointer (ptrtoint) to a global. This trips up the
-			// interp package. Therefore, make the id field a pointer for now.
-			typ = c.ctx.StructType([]llvm.Type{c.dataPtrType, c.dataPtrType}, false)
-		}
 		for i, subtyp := range typ.StructElementTypes() {
 			subptrs := c.getPointerBitmap(subtyp, pos)
 			if subptrs.BitLen() == 0 {
diff --git a/interp/memory.go b/interp/memory.go
index ba224f9d5..2812cd01c 100644
--- a/interp/memory.go
+++ b/interp/memory.go
@@ -824,19 +824,6 @@ func (v rawValue) rawLLVMValue(mem *memoryView) (llvm.Value, error) {
 			if err != nil {
 				return llvm.Value{}, err
 			}
-			if !field.IsAGlobalVariable().IsNil() {
-				elementType := field.GlobalValueType()
-				if elementType.TypeKind() == llvm.StructTypeKind {
-					// There are some special pointer types that should be used
-					// as a ptrtoint, so that they can be used in certain
-					// optimizations.
-					name := elementType.StructName()
-					if name == "runtime.funcValueWithSignature" {
-						uintptrType := ctx.IntType(int(mem.r.pointerSize) * 8)
-						field = llvm.ConstPtrToInt(field, uintptrType)
-					}
-				}
-			}
 			structFields = append(structFields, field)
 			i += mem.r.pointerSize
 			continue
diff --git a/src/runtime/func.go b/src/runtime/func.go
deleted file mode 100644
index 879424b5d..000000000
--- a/src/runtime/func.go
+++ /dev/null
@@ -1,28 +0,0 @@
-package runtime
-
-// This file implements some data types that may be useful for some
-// implementations of func values.
-
-import (
-	"unsafe"
-)
-
-// funcValue is the underlying type of func values, depending on which func
-// value representation was used.
-type funcValue struct {
-	context unsafe.Pointer // function context, for closures and bound methods
-	id      uintptr        // ptrtoint of *funcValueWithSignature before lowering, opaque index (non-0) after lowering
-}
-
-// funcValueWithSignature is used before the func lowering pass.
-type funcValueWithSignature struct {
-	funcPtr   uintptr // ptrtoint of the actual function pointer
-	signature *uint8  // external *i8 with a name identifying the function signature
-}
-
-// getFuncPtr is a dummy function that may be used if the func lowering pass is
-// not used. It is generally too slow but may be a useful fallback to debug the
-// func lowering pass.
-func getFuncPtr(val funcValue, signature *uint8) uintptr {
-	return (*funcValueWithSignature)(unsafe.Pointer(val.id)).funcPtr
-}