qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-17 06:04:47 -07:00
Code Releases Activity

cmd/compile: update comments with ABI aliases/wrappers

Browse Source 
ABI aliases are gone.

Change-Id: I0f5676d8730cac14b7495dd6c0c1e08ca85a1c77
Reviewed-on: https://go-review.googlesource.com/c/go/+/351531
Trust: Cherry Mui <cherryyz@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
This commit is contained in:
Cherry Mui 2021-09-22 13:23:47 -04:00
parent 1e57748892
commit 5b75a9b2c3
2 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/cmd/compile/internal/ssagen/abi.go
View File

@ -218,8 +218,6 @@ func (s *SymABIs) GenABIWrappers() {
}
if !buildcfg.Experiment.RegabiWrappers {
// We'll generate ABI aliases instead of
// wrappers once we have LSyms in InitLSym.
continue
}
@ -251,10 +249,9 @@ func InitLSym(f *ir.Func, hasBody bool) {
// the funcsym for either the defining
// function or its wrapper as appropriate.
//
// If we're using ABI aliases instead of
// wrappers, we only InitLSym for the defining
// ABI of a function, so we make the funcsym
// when we see that.
// If we're not using ABI wrappers, we only
// InitLSym for the defining ABI of a function,
// so we make the funcsym when we see that.
staticdata.NeedFuncSym(f)
}
}

2
src/cmd/compile/internal/staticdata/data.go
View File

@ -279,7 +279,7 @@ func NeedFuncSym(fn *ir.Func) {
// entry points, so it doesn't make sense to create a
// funcsym for other ABIs.
//
// (If we're using ABI aliases, it doesn't matter.)
// (If we're not using ABI wrappers, it doesn't matter.)
base.Fatalf("expected ABIInternal: %v has %v", fn.Nname, fn.ABI)
}
if ir.IsBlank(fn.Nname) {