diff options
author | Vinson Lee <vlee@freedesktop.org> | 2020-12-08 22:32:16 -0800 |
---|---|---|
committer | Marge Bot <eric+marge@anholt.net> | 2020-12-12 02:36:20 +0000 |
commit | 56cb5cf34e8edbf863ff5577efe851852472e9bd (patch) | |
tree | 2d434d0686b7902278ec7310bd52a609462c3ade | |
parent | 73c33803359dcb378c040abcc3f517fa4ea65646 (diff) |
intel/genxml: Avoid generating identical 12.5 and 12 branches.
Fix defects reported by Coverity Scan.
Identical code for different branches (IDENTICAL_BRANCHES)
identical_branches: The same code is executed regardless of
whether 0 is true, because the 'then' and 'else' branches are
identical. Should one of the branches be modified, or the entire
'if' statement replaced?
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8001>
-rw-r--r-- | src/intel/genxml/gen_bits_header.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/intel/genxml/gen_bits_header.py b/src/intel/genxml/gen_bits_header.py index f1a625add18..95443dc1ff7 100644 --- a/src/intel/genxml/gen_bits_header.py +++ b/src/intel/genxml/gen_bits_header.py @@ -81,11 +81,15 @@ ${item.token_name}_${prop}(const struct gen_device_info *devinfo) { switch (devinfo->gen) { case 12: +%if item.get_prop(prop, 12.5) == item.get_prop(prop, 12): + return ${item.get_prop(prop, 12)}; +%else: if (gen_device_info_is_12hp(devinfo)) { return ${item.get_prop(prop, 12.5)}; } else { return ${item.get_prop(prop, 12)}; } +%endif case 11: return ${item.get_prop(prop, 11)}; case 9: return ${item.get_prop(prop, 9)}; case 8: return ${item.get_prop(prop, 8)}; |