Mesa (main): pan/va: Fix MKVEC.v2i16 lane select

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Feb 2 18:00:11 UTC 2022


Module: Mesa
Branch: main
Commit: 46cd0ddcb67536245a64e1fdc5d92cb031b25b75
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=46cd0ddcb67536245a64e1fdc5d92cb031b25b75

Author: Alyssa Rosenzweig <alyssa at collabora.com>
Date:   Sat Jan 29 10:51:06 2022 -0500

pan/va: Fix MKVEC.v2i16 lane select

The lanes are at bit 28 and bit 26 respectively. This matches the 16-bit "swizzle" encoding. In general the handling of widens/swizzles/lane/lanes on Valhall is rather confused but... one problem at a time.

Signed-off-by: Alyssa Rosenzweig <alyssa at collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14833>

---

 src/panfrost/bifrost/valhall/ISA.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/panfrost/bifrost/valhall/ISA.xml b/src/panfrost/bifrost/valhall/ISA.xml
index 35cd9f5d470..01eab12ffc9 100644
--- a/src/panfrost/bifrost/valhall/ISA.xml
+++ b/src/panfrost/bifrost/valhall/ISA.xml
@@ -1456,8 +1456,8 @@
 
   <ins name="MKVEC.v2i16" title="Make 16-bit vector" dests="1" opcode="0xA1" opcode2="0x5" unit="CVT">
     <desc>Calculates $A | (B \ll 16)$. Used to implement `(ushort2)(A, B)`</desc>
-    <src widen="true">A</src>
-    <src widen="true">B</src>
+    <src swizzle="true">A</src>
+    <src swizzle="true">B</src>
   </ins>
 
   <group name="ISUB" title="Integer subtract" dests="1" opcode2="1" unit="CVT">



More information about the mesa-commit mailing list