[Mesa-dev] [PATCH 3/3] radv/ac: some fix maybe-uninitialized warnings

Grazvydas Ignotas notasas at gmail.com
Sat Dec 10 19:10:05 UTC 2016


Mark some paths unreachable so that compiler knows variables are
initialized in all valid paths.

Signed-off-by: Grazvydas Ignotas <notasas at gmail.com>
---
no commit access, somebody please push

 src/amd/common/ac_nir_to_llvm.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index d5dfbb1..398db6a 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@@ -721,7 +721,8 @@ build_gather_values_extended(struct nir_to_llvm_context *ctx,
 		if (load)
 			return LLVMBuildLoad(builder, values[0], "");
 		return values[0];
-	}
+	} else if (!value_count)
+		unreachable("value_count is 0");
 
 	for (i = 0; i < value_count; i++) {
 		LLVMValueRef value = values[i * value_stride];
@@ -3151,6 +3152,8 @@ static LLVMValueRef get_sampler_desc(struct nir_to_llvm_context *ctx,
 		type = ctx->v4i32;
 		type_size = 16;
 		break;
+	default:
+		unreachable("invalid desc_type\n");
 	}
 
 	if (deref->deref.child) {
-- 
2.7.4



More information about the mesa-dev mailing list