[Bindings][Go] Fixed go.test failure due to C-API argument type mismatch.

[Chirag]

Summary

go.test was failing previously with error,

Command Output (stderr):

llvm.org/llvm/bindings/go/llvm

/tmp/lit_tmp_LMr8rj/gopath207816655/src/llvm.org/llvm/bindings/go/llvm/dibuilder.go:301: cannot use C.uint(t.Encoding) (type C.uint) as type C.LLVMDWARFTypeEncoding in argument to func literal

this patch fixes the argument type.

[kristina]

Had the same issue when trying to build llgo and had to enable Go bindings, fixed in similar way locally.

[Chirag]

Hello Kristina,

I was unable to find any bug for it and as it was a minor fix, went for changes directly. If you are aware of the bug number can you kindly link it?

[kristina]

I haven't ever filed it, just locally fixed it as I was messing around with trying to build llgo correctly (check the mailing lists for llgo).

[pcc]

LGTM

[Chirag]

Can anyone kindly submit this? i do not have write access. Thanks.

[kristina]

Sure, one second.

bindings/go/llvm/dibuilder.go

 func (d *DIBuilder) CreateBasicType(t DIBasicType) Metadata {
   name := C.CString(t.Name)
   defer C.free(unsafe.Pointer(name))
   result := C.LLVMDIBuilderCreateBasicType(
     d.ref,
     name,
     C.size_t(len(t.Name)),
     C.uint64_t(t.SizeInBits),
-    C.unsigned(t.Encoding),
+    C.LLVMDWARFTypeEncoding(t.Encoding),
     C.LLVMDIFlags(0),
   )
   return Metadata{C: result}
 }
 
 // DIPointerType holds the values for creating pointer type debug metadata.
 type DIPointerType struct {
   Pointee     Metadata