anv: fix use of comma operator
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Tue, 23 Jul 2019 11:12:43 +0000 (14:12 +0300)
committerLionel Landwerlin <lionel.g.landwerlin@intel.com>
Tue, 23 Jul 2019 15:54:48 +0000 (15:54 +0000)
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

index 5660711692c0ddd9c31b50ca2b5eef697f75a697..9796ad29df0c78e9d69126b8854c0395400020ca 100644 (file)
@@ -1149,7 +1149,7 @@ emit_3dstate_clip(struct anv_pipeline *pipeline,
       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