From 8d453ed367f8891bd690cec0e7435cf0b05fb2ca Mon Sep 17 00:00:00 2001 From: Ayke van Laethem Date: Sun, 24 Jan 2021 16:48:55 +0100 Subject: [PATCH] compiler: remove unnecessary main.main call workaround Since https://github.com/tinygo-org/tinygo/pull/1571 (in particular, the first commit that sets the main package path), the main package is always named "main". This makes the callMain() workaround in the runtime unnecessary and allows directly calling the main.main function with a //go:linkname pragma. --- compiler/compiler.go | 8 -------- src/runtime/runtime.go | 3 +-- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/compiler/compiler.go b/compiler/compiler.go index 0ac4cd625a..a8a518253e 100644 --- a/compiler/compiler.go +++ b/compiler/compiler.go @@ -297,14 +297,6 @@ func CompileProgram(pkgName string, lprogram *loader.Program, machine llvm.Targe } irbuilder.CreateRetVoid() - // Conserve for goroutine lowering. Without marking these as external, they - // would be optimized away. - realMain := c.mod.NamedFunction(lprogram.MainPkg().Pkg.Path() + ".main") - realMain.SetLinkage(llvm.ExternalLinkage) // keep alive until goroutine lowering - - // Replace callMain placeholder with actual main function. - c.mod.NamedFunction("runtime.callMain").ReplaceAllUsesWith(realMain) - // Load some attributes getAttr := func(attrName string) llvm.Attribute { attrKind := llvm.AttributeKindID(attrName) diff --git a/src/runtime/runtime.go b/src/runtime/runtime.go index 3a4a8cd8d6..c648e4f2ce 100644 --- a/src/runtime/runtime.go +++ b/src/runtime/runtime.go @@ -10,8 +10,7 @@ const Compiler = "tinygo" // package. func initAll() -// callMain is a placeholder for the program main function. -// All references to this are replaced with references to the program main function by the compiler. +//go:linkname callMain main.main func callMain() func GOMAXPROCS(n int) int {