https://bugs.kde.org/show_bug.cgi?id=271615

--- Comment #17 from Mark Wielaard <[email protected]> ---
(In reply to mcermak from comment #15)
> Created attachment 192417 [details]
> updated patch
> 
> Apologies, one more update addressing your:
> > Nitpick: But since we have 32bit registers and only popcntw and popcntl
> > are supported I would have expected only using 32bit block values.

This looks good. Now it is easy to see what the popcnt operation really does
from the testcase.
Only nitpick (but feel free to ignore) you could just add a randUInt to
none/tests/sse4-common.h
But this (UInt) randULong also works. So feel free to commit this with or
without that.

> Also this patch splits out the updating of the cpuid part.

Lets do the cpuid update as a separate update once we figure out
how to handle lzcnt too (and maybe sse4.1 and sse4.2 - we have a plan
to finish x86 sse4.1, but not sse4.2).
https://en.wikipedia.org/wiki/Cpuid is the best reference for cpuid.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to