[PATCH v10 1/5] vkd3d-shader/ir: Refactor return code checking in vkd3d_shader_normalise().

[Conor McCarthy]

From: Conor McCarthy

We started with only one or two of these but it has become excessive.
---
 libs/vkd3d-shader/ir.c | 51 +++++++++++++++++++++++-------------------
 1 file changed, 28 insertions(+), 23 deletions(-)

diff --git a/libs/vkd3d-shader/ir.c b/libs/vkd3d-shader/ir.c
index 14288a38b..380fc0b92 100644
--- a/libs/vkd3d-shader/ir.c
+++ b/libs/vkd3d-shader/ir.c
@@ -2243,43 +2243,48 @@ enum vkd3d_result vkd3d_shader_normalise(struct vkd3d_shader_parser *parser,
         const struct vkd3d_shader_compile_info *compile_info)
 {
     struct vkd3d_shader_instruction_array *instructions = &parser->instructions;
-    enum vkd3d_result result = VKD3D_OK;
+    enum vkd3d_result result;

if (parser->shader_desc.is_dxil)
-        return result;
+        return VKD3D_OK;

-    if (parser->shader_version.type != VKD3D_SHADER_TYPE_PIXEL
-            && (result = remap_output_signature(parser, compile_info)) < 0)
-        return result;
+    if (parser->shader_version.type != VKD3D_SHADER_TYPE_PIXEL)
+    {
+        if ((result = remap_output_signature(parser, compile_info)) < 0)
+            return result;
+    }

-    if (parser->shader_version.type == VKD3D_SHADER_TYPE_HULL
-            && (result = instruction_array_flatten_hull_shader_phases(instructions)) >= 0)
+    if (parser->shader_version.type == VKD3D_SHADER_TYPE_HULL)
     {
-        result = instruction_array_normalise_hull_shader_control_point_io(instructions,
-                &parser->shader_desc.input_signature);
+        if ((result = instruction_array_flatten_hull_shader_phases(instructions)) < 0)
+            return result;
+
+        if ((result = instruction_array_normalise_hull_shader_control_point_io(instructions,
+                &parser->shader_desc.input_signature)) < 0)
+            return result;
     }
-    if (result >= 0)
-        result = shader_normalise_io_registers(parser);

-    if (result >= 0)
-        result = instruction_array_normalise_flat_constants(parser);
+    if ((result = shader_normalise_io_registers(parser)) < 0)
+        return result;

-    if (result >= 0)
-        remove_dead_code(parser);
+    if ((result = instruction_array_normalise_flat_constants(parser)) < 0)
+        return result;

-    if (result >= 0)
-        result = flatten_control_flow_constructs(parser);
+    remove_dead_code(parser);

-    if (result >= 0)
-        result = normalise_combined_samplers(parser);
+    if ((result = flatten_control_flow_constructs(parser)) < 0)
+        return result;
+
+    if ((result = normalise_combined_samplers(parser)) < 0)
+        return result;

-    if (result >= 0 && TRACE_ON())
+    if (TRACE_ON())
         vkd3d_shader_trace(instructions, &parser->shader_version);

-    if (result >= 0 && !parser->failed)
-        result = vsir_validate(parser);
+    if (!parser->failed && (result = vsir_validate(parser)) < 0)
+        return result;

-    if (result >= 0 && parser->failed)
+    if (parser->failed)
         result = VKD3D_ERROR_INVALID_SHADER;

return result;