• isharipo's avatar
    runtime,sync/atomic: replace asm BYTEs with insts for x86 · ff5cf43d
    isharipo authored
    For each replacement, test case is added to new 386enc.s file
    with exception of EMMS, SYSENTER, MFENCE and LFENCE as they
    are already covered in amd64enc.s (same on amd64 and 386).
    
    The replacement became less obvious after go vet suggested changes
    Before:
    	BYTE $0x0f; BYTE $0x7f; BYTE $0x44; BYTE $0x24; BYTE $0x08
    Changed to MOVQ (this form is being tested):
    	MOVQ M0, 8(SP)
    Refactored to FP-relative access (go vet advice):
    	MOVQ M0, val+4(FP)
    
    Change-Id: I56b87cf3371b6ad81ad0cd9db2033aee407b5818
    Reviewed-on: https://go-review.googlesource.com/101475
    Run-TryBot: Iskander Sharipov <iskander.sharipov@intel.com>
    TryBot-Result: Gobot Gobot <gobot@golang.org>
    Reviewed-by: 's avatarIlya Tocar <ilya.tocar@intel.com>
    ff5cf43d
asm_386.s 3.87 KB