[Qemu-devel] [Bug 591320] Re: [ARM]: SIMD add/sub instructions are incorrect

2010-07-28 Thread cmchao
The patch has been included in v0.12.5 ** Changed in: qemu Status: New => Fix Released -- [ARM]: SIMD add/sub instructions are incorrect https://bugs.launchpad.net/bugs/591320 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. Stat

[Qemu-devel] [Bug 591320] Re: [ARM]: SIMD add/sub instructions are incorrect

2010-06-09 Thread cmchao
One attachment is allowed in each comment :< for case 1 ** Patch added: "for case 1 problem" http://launchpadlibrarian.net/5666/0002-target-arm-fix-parallel-saturated-subtraction-error.patch -- [ARM]: SIMD add/sub instructions are incorrect https://bugs.launchpad.net/bugs/591320 You rec

[Qemu-devel] [Bug 591320] Re: [ARM]: SIMD add/sub instructions are incorrect

2010-06-09 Thread cmchao
The first one results from wrong saturation judgment. The original form is : uint8_t sub8_usat(uint8_t a, uint8_t b) { if (a < b) return a - b; else return 0; } it should be " a > b" The second problem is using op2 as op1 to tell among signed, unsigned, and half group