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

Author: Kenneth Graunke <kenn...@whitecape.org>
Date:   Fri Dec  9 01:33:25 2022 -0800

intel/compiler: Assert that FS_OPCODE_[REP_]FB_WRITE is for pre-Gfx7

We use SHADER_OPCODE_SEND directly instead of FS_OPCODE_FB_WRITE (for a
while now) and FS_OPCODE_REP_FB_WRITE (since the previous commit).

Assert that it isn't used on Gfx7+.

Reviewed-by: Caio Oliveira <caio.olive...@intel.com>
Reviewed-by: Emma Anholt <e...@anholt.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20172>

---

 src/intel/compiler/brw_fs_generator.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/intel/compiler/brw_fs_generator.cpp 
b/src/intel/compiler/brw_fs_generator.cpp
index 0c1d1fc6a57..44144597ee8 100644
--- a/src/intel/compiler/brw_fs_generator.cpp
+++ b/src/intel/compiler/brw_fs_generator.cpp
@@ -401,10 +401,10 @@ fs_generator::fire_fb_write(fs_inst *inst,
 void
 fs_generator::generate_fb_write(fs_inst *inst, struct brw_reg payload)
 {
-   if (devinfo->verx10 <= 70) {
-      brw_set_default_predicate_control(p, BRW_PREDICATE_NONE);
-      brw_set_default_flag_reg(p, 0, 0);
-   }
+   assert(devinfo->ver < 7);
+
+   brw_set_default_predicate_control(p, BRW_PREDICATE_NONE);
+   brw_set_default_flag_reg(p, 0, 0);
 
    const struct brw_reg implied_header =
       devinfo->ver < 6 ? payload : brw_null_reg();

Reply via email to