diff options
| author | Ian Lance Taylor <iant@golang.org> | 2010-05-30 22:22:47 -0700 |
|---|---|---|
| committer | Ian Lance Taylor <iant@golang.org> | 2010-05-30 22:22:47 -0700 |
| commit | b51989bcee8aa7e0742bbcf15a2940aa04f9f244 (patch) | |
| tree | d9898f7570425a4ddf009de3ae2def4a60cf14db /src/cmd | |
| parent | bfa103a2fe1fe2872e6539527d1108a501c2f1af (diff) | |
| download | golang-b51989bcee8aa7e0742bbcf15a2940aa04f9f244.tar.gz | |
Correct _cgo_free when C ABI does not pass first arg on stack.
It turns out that _cgo_malloc is used, via cmalloc in
runtime/cgocall.c, which is called by code generated by out.go
for the ·_C_CString function. I can't find a call to
_cgo_free, but given _cgo_malloc we might as well keep
_cgo_free. This patch fixes it so that it should work on
amd64.
R=rsc
CC=golang-dev
http://codereview.appspot.com/1399041
Diffstat (limited to 'src/cmd')
| -rw-r--r-- | src/cmd/cgo/out.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/cgo/out.go b/src/cmd/cgo/out.go index 2fae48fe4..e58923ab2 100644 --- a/src/cmd/cgo/out.go +++ b/src/cmd/cgo/out.go @@ -568,7 +568,7 @@ const cProlog = ` #pragma dynimport libcgo_thread_start libcgo_thread_start "%s/libcgo.so" #pragma dynimport libcgo_set_scheduler libcgo_set_scheduler "%s/libcgo.so" #pragma dynimport _cgo_malloc _cgo_malloc "%s/libcgo.so" -#pragma dynimport _cgo_free free "%s/libcgo.so" +#pragma dynimport _cgo_free _cgo_free "%s/libcgo.so" void ·_C_GoString(int8 *p, String s) |