[llvm] f337d58 - [OCaml] Fix warnings in llvm_debuginfo (#173011)

[via llvm-commits]

Author: Nikita Popov
Date: 2025-12-22T10:08:49+01:00
New Revision: f337d5843fd2a5017665eceeb968756ccbb442ff

URL: https://github.com/llvm/llvm-project/commit/f337d5843fd2a5017665eceeb968756ccbb442ff
DIFF: https://github.com/llvm/llvm-project/commit/f337d5843fd2a5017665eceeb968756ccbb442ff.diff

LOG: [OCaml] Fix warnings in llvm_debuginfo (#173011)

Fix an unused variable warning and a warning about missing return. As
this is C code we don't have llvm_unreachable() here, so I went with an
assert + dummy return.

Added: 
    

Modified: 
    llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c

Removed: 
    


################################################################################
diff  --git a/llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c b/llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c
index 023ebd6d60cd5..26289d785ebd2 100644
--- a/llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c
+++ b/llvm/bindings/ocaml/debuginfo/debuginfo_ocaml.c
@@ -15,6 +15,7 @@
 |*                                                                            *|
 \*===----------------------------------------------------------------------===*/
 
+#include <assert.h>
 #include <string.h>
 
 #include "caml/memory.h"
@@ -137,6 +138,8 @@ static LLVMDIFlags map_DIFlag(LLVMDIFlag_i DIF) {
   case i_DIFlagPtrToMemberRep:
     return LLVMDIFlagPtrToMemberRep;
   }
+  assert(0 && "Invalid LLVMDIFlag");
+  return 0;
 }
 
 /* unit -> int */
@@ -380,7 +383,6 @@ value llvm_dibuild_get_or_create_array(value Builder, value Data) {
 
 value llvm_dibuild_create_subroutine_type(value Builder, value File,
                                           value ParameterTypes, value Flags) {
-  mlsize_t Count = Wosize_val(ParameterTypes);
   LLVMMetadataRef *Temp = from_val_array(ParameterTypes);
   LLVMMetadataRef Metadata = LLVMDIBuilderCreateSubroutineType(
       DIBuilder_val(Builder), Metadata_val(File), Temp,