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

Carl Love <c...@us.ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |CONFIRMED

--- Comment #3 from Carl Love <c...@us.ibm.com> ---
Yes, it is changing the semantics.  The code a implemented is wrong.  The field
for the steering value is 5 bits according to the ISA, not four as implemented.
 So values greater then 16 are not handled correctly.

The ISA says:

Let the source vector be the concatenation of the
contents of VR[VRA] followed by the contents of
VR[VRB].

For each integer value i from 0 to 15, do the following.
  Let index be the value specified by bits 3:7 of byte
  element i of VR[VRC].

So, the index value is 5-bits wide.

The code for the vperm instruction was reused with some small tweaks to
implement the vpermr, xxperm and xxpermr instructions.  In testing the code for
these other instructions this issue was found. 

Probably should have separated it out into its own bug fix.

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

Reply via email to