use llvm EmitToFile for native object emission

Author: cpunion

go.mod

@@ -28,3 +28,5 @@ require (
 )
 
 replace github.com/goplus/llgo/runtime => ./runtime
+
+replace github.com/goplus/llvm => github.com/cpunion/llvm v0.8.9-0.20260429084913-40fdafa22ac4

go.sum

@@ -1,3 +1,5 @@
+github.com/cpunion/llvm v0.8.9-0.20260429084913-40fdafa22ac4 h1:R4uLegmI8JT2Dg3BBsHfGtfXVIm0HvGEGuG4RJLqvEU=
+github.com/cpunion/llvm v0.8.9-0.20260429084913-40fdafa22ac4/go.mod h1:PeVK8GgzxwAYCiMiUAJb5wJR6xbhj989tu9oulKLLT4=
 github.com/creack/goselect v0.1.2 h1:2DNy14+JPjRBgPzAd1thbQp4BSIihxcBf0IXhQXDRa0=
 github.com/creack/goselect v0.1.2/go.mod h1:a/NhLweNvqIYMuxcMOuWY516Cimucms3DglDzQP3hKY=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
@@ -10,8 +12,6 @@ github.com/goplus/gogen v1.22.0 h1:clM2eMVWQZz7tYUU/u9xwKFCBLcyVbcstzdiZrgRdfA=
 github.com/goplus/gogen v1.22.0/go.mod h1:Y7ulYW3wonQ3d9er00b0uGFEV/IUZa6okWJZh892ACQ=
 github.com/goplus/lib v0.3.1 h1:Xws4DBVvgOMu58awqB972wtvTacDbk3nqcbHjdx9KSg=
 github.com/goplus/lib v0.3.1/go.mod h1:SgJv3oPqLLHCu0gcL46ejOP3x7/2ry2Jtxu7ta32kp0=
-github.com/goplus/llvm v0.8.8 h1:02GfPE54wVlYtahedzXXFl0JHND8Y+kH6Ng3iHL5BOo=
-github.com/goplus/llvm v0.8.8/go.mod h1:PeVK8GgzxwAYCiMiUAJb5wJR6xbhj989tu9oulKLLT4=
 github.com/goplus/mod v0.20.2 h1:YX72E6AhhRLvlkVnI9cBK6PZvUwtge2hwROh7w9N6Yk=
 github.com/goplus/mod v0.20.2/go.mod h1:lWW62tH7L3Vm42Lr6wlUMYHvsm5w3TkEpE2ulKTgmU8=
 github.com/goplus/plan9asm v0.2.1 h1:XI9OoxyBmjhE/R81m3jHnhkZDMdl+t2OHn9iwozcXV8=

internal/build/build.go

@@ -1438,17 +1438,17 @@ func buildPkg(ctx *context, aPkg *aPackage, verbose bool) error {
 }
 
 func exportObject(ctx *context, pkgPath string, exportFile string, pkg llssa.Package) (string, error) {
-	if useInMemoryNativeCodegen(ctx) {
+	if useNativeLLVMCodegen(ctx) {
 		return exportObjectInMemory(ctx, pkgPath, exportFile, pkg)
 	}
 	return exportObjectWithClang(ctx, pkgPath, exportFile, []byte(pkg.String()))
 }
 
-func useInMemoryNativeCodegen(ctx *context) bool {
-	return useInMemoryNativeCodegenConf(ctx.buildConf)
+func useNativeLLVMCodegen(ctx *context) bool {
+	return useNativeLLVMCodegenConf(ctx.buildConf)
 }
 
-func useInMemoryNativeCodegenConf(conf *Config) bool {
+func useNativeLLVMCodegenConf(conf *Config) bool {
 	return conf != nil && !conf.GenLL &&
 		conf.Target == "" &&
 		conf.Goos == runtime.GOOS &&
@@ -1500,28 +1500,22 @@ func exportObjectInMemory(ctx *context, pkgPath string, exportFile string, pkg l
 			return "", err
 		}
 	}
-	buf, err := ctx.prog.TargetMachine().EmitToMemoryBuffer(pkg.Module(), gllvm.ObjectFile)
-	if err != nil {
-		return "", err
-	}
-	defer buf.Dispose()
 
 	base := filepath.Base(exportFile)
 	objFile, err := os.CreateTemp("", base+"-*.o")
 	if err != nil {
 		return "", err
 	}
 	objFileName := objFile.Name()
-	if ctx.shouldPrintCommands(false) {
-		fmt.Fprintf(os.Stderr, "# compiling %s for pkg: %s\n", objFileName, pkgPath)
-		fmt.Fprintln(os.Stderr, "# using in-memory LLVM object emission")
-	}
-	if _, err := objFile.Write(buf.Bytes()); err != nil {
-		objFile.Close()
+	if err := objFile.Close(); err != nil {
 		os.Remove(objFileName)
 		return "", err
 	}
-	if err := objFile.Close(); err != nil {
+	if ctx.shouldPrintCommands(false) {
+		fmt.Fprintf(os.Stderr, "# compiling %s for pkg: %s\n", objFileName, pkgPath)
+		fmt.Fprintln(os.Stderr, "# using direct LLVM object emission")
+	}
+	if err := ctx.prog.TargetMachine().EmitToFile(pkg.Module(), objFileName, gllvm.ObjectFile); err != nil {
 		os.Remove(objFileName)
 		return "", err
 	}

internal/build/llvm_emit_test.go

@@ -5,17 +5,17 @@ import (
 	"testing"
 )
 
-func TestUseInMemoryNativeCodegenConf(t *testing.T) {
+func TestUseNativeLLVMCodegenConf(t *testing.T) {
 	t.Run("native host", func(t *testing.T) {
 		conf := &Config{Goos: runtime.GOOS, Goarch: runtime.GOARCH}
-		if !useInMemoryNativeCodegenConf(conf) {
-			t.Fatal("expected native host build to use in-memory native codegen")
+		if !useNativeLLVMCodegenConf(conf) {
+			t.Fatal("expected native host build to use native LLVM codegen")
 		}
 	})
 
 	t.Run("embedded target", func(t *testing.T) {
 		conf := &Config{Goos: runtime.GOOS, Goarch: runtime.GOARCH, Target: "rp2040"}
-		if useInMemoryNativeCodegenConf(conf) {
+		if useNativeLLVMCodegenConf(conf) {
 			t.Fatal("expected embedded target build to keep using clang")
 		}
 	})
@@ -34,14 +34,14 @@ func TestUseInMemoryNativeCodegenConf(t *testing.T) {
 			goarch = "amd64"
 		}
 		conf := &Config{Goos: goos, Goarch: goarch}
-		if useInMemoryNativeCodegenConf(conf) {
+		if useNativeLLVMCodegenConf(conf) {
 			t.Fatal("expected host mismatch to keep using clang")
 		}
 	})
 
 	t.Run("wasm", func(t *testing.T) {
 		conf := &Config{Goos: "wasip1", Goarch: "wasm"}
-		if useInMemoryNativeCodegenConf(conf) {
+		if useNativeLLVMCodegenConf(conf) {
 			t.Fatal("expected wasm target to keep using clang")
 		}
 	})