Bug 405733 - PPC64, xvcvdpsp should write 32-bit result to upper and lower 32-bits of the 64-bit destination field.
Summary: PPC64, xvcvdpsp should write 32-bit result to upper and lower 32-bits of the ...
Status: CLOSED FIXED
Alias: None
Product: valgrind
Classification: Developer tools
Component: vex (other bugs)
Version First Reported In: 3.15 SVN
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Julian Seward
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-03-21 21:13 UTC by Carl Love
Modified: 2019-03-25 14:48 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
fix for the xvcvdpsp instruction (3.22 KB, patch)
2019-03-22 17:41 UTC, Carl Love
Details
version 2, a line got left out of the previous patch (3.30 KB, patch)
2019-03-22 17:46 UTC, Carl Love
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Carl Love 2019-03-21 21:13:08 UTC
The xvcvdpsp instruction currently only writes the result to the lower 32-bits of the destination field.  It should write the same result to the upper and lower 32-bits of the result.
Comment 1 Carl Love 2019-03-22 17:41:27 UTC
Created attachment 118975 [details]
fix for the xvcvdpsp instruction

The attached patch was committed to fix the issue.

commit 91bd56be4153cb5cf6d1fa80ade4b2432ff81ab0
Author: Carl Love <carll@us.ibm.com>
Date:   Fri Mar 22 12:39:32 2019 -0500
Comment 2 Carl Love 2019-03-22 17:46:12 UTC
Created attachment 118976 [details]
version 2, a line got left out of the previous patch

Updated patch, caught the missing line before pushing patch to mainline.

commit 30a24515f09f9118c469ae11c9d574da729c97ed
Author: Carl Love <carll@us.ibm.com>
Date:   Fri Mar 22 12:42:27 2019 -0500
Comment 3 Carl Love 2019-03-25 14:48:20 UTC
No regressions found by automated regression testing.  Closing.