commit 772a5f981485d81919f1cb8ab52ddff0412c6ee5
author Lionel Landwerlin <lionel.g.landwerlin@intel.com> Tue Jul 23 14:12:43 2019 +0300
committer Lionel Landwerlin <lionel.g.landwerlin@intel.com> Tue Jul 23 15:54:48 2019 +0000
tree 8ef948b3e6baf3cc2f5b477d23850fd784527524
parent 79ab2c3e5764f6ac5e20338dc2c6c750eed9abd2

anv: fix use of comma operator

This doesn't fix any bug at the moment because the next statement is
'true' which happens to be APIMODE_D3D, but if that changes it could.

The fixes tags is as far I could go but the error predates it (2016 is
probably far enough).

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 8db6f2e6ebb9 ("anv/pipeline: Roll genX_pipeline_util.h into genX_pipeline.c")
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>

src/intel/vulkan/genX_pipeline.c

diff --git a/src/intel/vulkan/genX_pipeline.c b/src/intel/vulkan/genX_pipeline.c
index 5660711..9796ad2 100644
--- a/src/intel/vulkan/genX_pipeline.c
+++ b/src/intel/vulkan/genX_pipeline.c
@@ -1149,7 +1149,7 @@
       clip.ClipEnable               = true;
       clip.StatisticsEnable         = true;
       clip.EarlyCullEnable          = true;
-      clip.APIMode                  = APIMODE_D3D,
+      clip.APIMode                  = APIMODE_D3D;
       clip.GuardbandClipTestEnable  = true;
 
       /* Only enable the XY clip test when the final polygon rasterization