Mesa (master): aco: use FALLTHROUGH macro

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Tue Dec 1 10:44:00 UTC 2020


Module: Mesa
Branch: master
Commit: 2d12991e016bbb948c01a86e7628440987c244c2
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2d12991e016bbb948c01a86e7628440987c244c2

Author: Rhys Perry <pendingchaos02 at gmail.com>
Date:   Tue Dec  1 09:54:31 2020 +0000

aco: use FALLTHROUGH macro

Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7844>

---

 src/amd/compiler/aco_instruction_selection.cpp       | 3 +--
 src/amd/compiler/aco_instruction_selection_setup.cpp | 2 +-
 src/amd/compiler/aco_optimizer.cpp                   | 6 +++---
 src/amd/compiler/aco_validate.cpp                    | 2 +-
 4 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/amd/compiler/aco_instruction_selection.cpp b/src/amd/compiler/aco_instruction_selection.cpp
index 58bffb8b70b..f1621b231cb 100644
--- a/src/amd/compiler/aco_instruction_selection.cpp
+++ b/src/amd/compiler/aco_instruction_selection.cpp
@@ -7688,8 +7688,7 @@ void visit_intrinsic(isel_context *ctx, nir_intrinsic_instr *instr)
          bld.copy(Definition(dst), Operand(get_arg(ctx, ctx->args->ac.view_index)));
          break;
       }
-
-      /* fallthrough */
+      FALLTHROUGH;
    }
    case nir_intrinsic_load_layer_id: {
       unsigned idx = nir_intrinsic_base(instr);
diff --git a/src/amd/compiler/aco_instruction_selection_setup.cpp b/src/amd/compiler/aco_instruction_selection_setup.cpp
index 008db0a68e9..94053a0b9df 100644
--- a/src/amd/compiler/aco_instruction_selection_setup.cpp
+++ b/src/amd/compiler/aco_instruction_selection_setup.cpp
@@ -738,7 +738,7 @@ void init_context(isel_context *ctx, nir_shader *shader)
                      break;
                   case nir_op_bcsel:
                      type = nir_dest_is_divergent(alu_instr->dest.dest) ? RegType::vgpr : RegType::sgpr;
-                     /* fallthrough */
+                     FALLTHROUGH;
                   default:
                      for (unsigned i = 0; i < nir_op_infos[alu_instr->op].num_inputs; i++) {
                         if (regclasses[alu_instr->src[i].src.ssa->index].type() == RegType::vgpr)
diff --git a/src/amd/compiler/aco_optimizer.cpp b/src/amd/compiler/aco_optimizer.cpp
index 30005394fee..123963296d4 100644
--- a/src/amd/compiler/aco_optimizer.cpp
+++ b/src/amd/compiler/aco_optimizer.cpp
@@ -1263,7 +1263,7 @@ void label_instruction(opt_ctx &ctx, Block& block, aco_ptr<Instruction>& instr)
          ctx.info[instr->definitions[0].tempId()].set_vec(instr.get());
          break;
       }
-      /* fallthrough */
+      FALLTHROUGH;
    case aco_opcode::p_as_uniform:
       if (instr->definitions[0].isFixed()) {
          /* don't copy-propagate copies into fixed registers */
@@ -1458,7 +1458,7 @@ void label_instruction(opt_ctx &ctx, Block& block, aco_ptr<Instruction>& instr)
             }
          }
       }
-      /* fallthrough */
+      FALLTHROUGH;
    case aco_opcode::s_or_b32:
    case aco_opcode::s_or_b64:
    case aco_opcode::s_xor_b32:
@@ -1468,7 +1468,7 @@ void label_instruction(opt_ctx &ctx, Block& block, aco_ptr<Instruction>& instr)
                       })) {
          ctx.info[instr->definitions[0].tempId()].set_uniform_bitwise();
       }
-      /* fallthrough */
+      FALLTHROUGH;
    case aco_opcode::s_lshl_b32:
    case aco_opcode::v_or_b32:
    case aco_opcode::v_lshlrev_b32:
diff --git a/src/amd/compiler/aco_validate.cpp b/src/amd/compiler/aco_validate.cpp
index eb351d5661a..ae7e8a93ed2 100644
--- a/src/amd/compiler/aco_validate.cpp
+++ b/src/amd/compiler/aco_validate.cpp
@@ -467,7 +467,7 @@ bool validate_ir(Program* program)
          }
          case Format::FLAT:
             check(instr->operands[1].isUndefined(), "Flat instructions don't support SADDR", instr.get());
-            /* fallthrough */
+            FALLTHROUGH;
          case Format::GLOBAL:
          case Format::SCRATCH: {
             check(instr->operands[0].isTemp() && instr->operands[0].regClass().type() == RegType::vgpr, "FLAT/GLOBAL/SCRATCH address must be vgpr", instr.get());



More information about the mesa-commit mailing list