commit f820dde201f82763b5165dfe6516f20e9c80795b
author Timur Kristóf <timur.kristof@gmail.com> Sat Aug 22 20:45:54 2020 +0200
committer Timur Kristóf <timur.kristof@gmail.com> Mon Aug 24 15:55:14 2020 +0200
tree 4ef97115020d3ec155658dc4a6b2aef2e9600e25
parent 0d194a70c6497ea77bb57aa22e229c4fceabb653

aco: Fix convert_to_SDWA when instruction has 3 operands.

Previously, when the instruction had 3 operands, this would cause
possible corruption because of writing to sdwa->sel[2].
This was noticed thanks to GCC 10's -Wstringop-overflow warning.

Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6436>

src/amd/compiler/aco_ir.cpp

diff --git a/src/amd/compiler/aco_ir.cpp b/src/amd/compiler/aco_ir.cpp
index 801fbc6..b594b282 100644
--- a/src/amd/compiler/aco_ir.cpp
+++ b/src/amd/compiler/aco_ir.cpp
@@ -238,6 +238,10 @@
    }
 
    for (unsigned i = 0; i < instr->operands.size(); i++) {
+      /* SDWA only uses operands 0 and 1. */
+      if (i >= 2)
+         break;
+
       switch (instr->operands[i].bytes()) {
       case 1:
          sdwa->sel[i] = sdwa_ubyte;