diff options
author | Kenneth Graunke <[email protected]> | 2019-04-03 14:24:31 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2019-04-28 22:36:09 -0700 |
commit | 9dcf90d7ba966cba904266209087a87ea9c018b9 (patch) | |
tree | 3abf4f33ff026f1aeac7a4c3f077b3273ee5bf50 /src/intel/compiler/brw_fs_nir.cpp | |
parent | 2b44b27dbe994a39d511211dcce1bb8367b0d066 (diff) |
intel/fs: Don't emit empty ELSE blocks.
While we can clean this up later, it's trivial to not generate the
stupid code in the first place, which saves some optimization work.
Reviewed-by: Jason Ekstrand <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/intel/compiler/brw_fs_nir.cpp')
-rw-r--r-- | src/intel/compiler/brw_fs_nir.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index 71fe0ff7e2f..526cc13c71a 100644 --- a/src/intel/compiler/brw_fs_nir.cpp +++ b/src/intel/compiler/brw_fs_nir.cpp @@ -409,10 +409,10 @@ fs_visitor::nir_emit_if(nir_if *if_stmt) nir_emit_cf_list(&if_stmt->then_list); - /* note: if the else is empty, dead CF elimination will remove it */ - bld.emit(BRW_OPCODE_ELSE); - - nir_emit_cf_list(&if_stmt->else_list); + if (!nir_cf_list_is_empty_block(&if_stmt->else_list)) { + bld.emit(BRW_OPCODE_ELSE); + nir_emit_cf_list(&if_stmt->else_list); + } bld.emit(BRW_OPCODE_ENDIF); |