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

Mark Wielaard <m...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |m...@redhat.com

--- Comment #3 from Mark Wielaard <m...@redhat.com> ---
(In reply to Axel Müller from comment #2)
> (In reply to Julian Seward from comment #1)
> > f2 49 0f 5d 00              rex.WB minsd (%r8),%xmm0
> > f2 49 0f 5f 00              rex.WB maxsd (%r8),%xmm0
> > 
> > I'm sure these insns are handled really.  It's just the redundant rex.WB
> > prefix
> > that is causing them not to get decoded.
> So what would you suggest? To be honest I only modified another patch
> without really understanding the code.

I think you patch is ok. Maybe you could say:

      if (haveF2no66noF3(pfx)
          && (sz == 4
                 || /* ignore redundant REX.W */ sz == 8 && getRexW(pfx) == 1))
{

But we would need a testcase. What generates these minsd/maxsd with extra
REX.W?

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

Reply via email to