I'm not super confident in these changes. * I changed all the conversions in decode.go ...
9 years, 10 months ago
(2014-06-11 01:14:33 UTC)
#2
I'm not super confident in these changes.
* I changed all the conversions in decode.go to sign-extend, but I wasn't able
to confirm that all the underlying values are in fact signed. I hoped that the
unit tests would clarify, or that you would know offhand.
* I can't run the full unit tests due to not having the requisite binaries.
* The short unit tests were failing before this CL:
--- FAIL: TestDecode (0.02 seconds)
decode_test.go:68: Decode(3667f3660ff7c0|11223344556677885f) [gnu] = ss, 1,
want ss addr32 rep maskmovdqu %xmm0,%xmm0, 7
Issue 100480046: code review 100480046: x86asm: sign-extend values but print them unsigned
(Closed)
Created 9 years, 11 months ago by josharian
Modified 9 years, 3 months ago
Reviewers: rsc
Base URL:
Comments: 0