From: Richard Miller Date: Wed, 27 Jan 2016 19:10:11 +0000 (+0000) Subject: runtime: remove redundant empty function call from Breakpoint on arm X-Git-Tag: go1.6rc1~9 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=d326a9641994eccdac1c95901762af45ec801bf1;p=gostls13.git runtime: remove redundant empty function call from Breakpoint on arm CL 18964 included an extra patch (sorry, my first experience of git-codereview) which defined the conventional breakpoint instruction used by Plan 9 on arm, but also introduced a benign but unneeded call to runtime.emptyfunc. This CL removes the redundant call again. This completes the series of CLs which add support for Plan 9 on arm. Change-Id: Id293cfd40557c9d79b4b6cb164ed7ed49295b178 Reviewed-on: https://go-review.googlesource.com/19010 Reviewed-by: Brad Fitzpatrick --- diff --git a/src/runtime/asm_arm.s b/src/runtime/asm_arm.s index 53128e7eb1..07894a3a72 100644 --- a/src/runtime/asm_arm.s +++ b/src/runtime/asm_arm.s @@ -81,7 +81,6 @@ DATA runtime·mainPC+0(SB)/4,$runtime·main(SB) GLOBL runtime·mainPC(SB),RODATA,$4 TEXT runtime·breakpoint(SB),NOSPLIT,$0-0 - BL runtime·emptyfunc(SB) // force R14 save for traceback // gdb won't skip this breakpoint instruction automatically, // so you must manually "set $pc+=4" to skip it and continue. #ifdef GOOS_nacl