Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 301780 - dev-util/valgrind: add support or the "lzcnt" instruction
Summary: dev-util/valgrind: add support or the "lzcnt" instruction
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Anthony Basile
URL: https://bugs.kde.org/show_bug.cgi?id=...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-21 23:29 UTC by Gert Wollny
Modified: 2011-09-17 21:40 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Patch to make valgrind aware of the sse4a lzcnt instruction (valgrind-3.5.0-amd64-lzcnt.patch,6.47 KB, text/plain)
2010-01-21 23:30 UTC, Gert Wollny 		Details
Patch to apply lzcnt patch in the ebuild (valgrind-3.5.0-to-r1.patch,461 bytes, text/plain)
2010-01-21 23:33 UTC, Gert Wollny 		Details
Revised patch that sets the carry flag properly (valgrind-3.5.0-amd64-lzcnt.patch,8.73 KB, patch)
2010-01-23 00:43 UTC, Gert Wollny 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gert Wollny 2010-01-21 23:29:26 UTC 
On Gentoo AMD64 compiled with -march=amdfam10 valgrind may fail with 
  vex amd64->IR: unhandled instruction bytes: 0xF3 0x48 0xF 0xBD 0xC0 0x4C
Attached patch fixes valgrind to recognize and execute the instruction.
Note that the patch is experimental and will be reviewed on the valgrind-dev mailing list. 
 


Reproducible: Always
Comment 1 Gert Wollny 2010-01-21 23:30:16 UTC 
Created attachment 217121 [details]
Patch to make valgrind aware of the sse4a lzcnt instruction
Comment 2 Gert Wollny 2010-01-21 23:33:07 UTC 
Created attachment 217123 [details]
Patch to apply lzcnt patch in the ebuild

Building the test of the sse4a instruction will not be enabled because it would require running autotools which is not yet in the ebuild.
Comment 3 Gert Wollny 2010-01-23 00:43:50 UTC 
Created attachment 217196 [details, diff]
Revised patch that sets the carry flag properly
Comment 4 marbacz 2010-04-02 14:14:36 UTC 
Patch tested on AMD Phenom. Works well.
Comment 5 Gert Wollny 2011-09-17 21:40:01 UTC 
Fixed upstream as of version 3.6.0.