pvr: do not use fallthrough for unreachable code

unreachable() doesn't lead to executing the code that follows it,
neither in debug nor release builds. So falling through doesn't make any
sense.

This fixes a compile-error on clang.

Let's move the default-block to the end to make it clearer that there's
no intended fallthrough.

Reviewed-by: Frank Binns <frank.binns@imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15531>

diff --git a/src/imagination/include/hwdef/rogue_hw_utils.h b/src/imagination/include/hwdef/rogue_hw_utils.h
index fc1e4e6..39e3bea 100644 (file)
--- a/src/imagination/include/hwdef/rogue_hw_utils.h
+++ b/src/imagination/include/hwdef/rogue_hw_utils.h
@@ -75,9 +75,6 @@ rogue_get_isp_samples_per_tile_xy(const struct pvr_device_info *dev_info,
    *y_out = tile_size_y;
 
    switch (samples) {
-   default:
-      assert(!"Unsupported number of samples");
-      FALLTHROUGH;
    case 1:
       break;
    case 2:
@@ -96,6 +93,8 @@ rogue_get_isp_samples_per_tile_xy(const struct pvr_device_info *dev_info,
    case 8:
       *y_out *= 2;
       break;
+   default:
+      assert(!"Unsupported number of samples");
    }
 }