Revert "Create a safe wrapper around LLVMRustDIBuilderCreateBasicType"

This reverts commit 018032c.

Author: petrochenkov

compiler/rustc_codegen_llvm/src/debuginfo/metadata.rs

@@ -39,8 +39,8 @@ use crate::debuginfo::metadata::type_map::build_type_with_children;
 use crate::debuginfo::utils::{WidePtrKind, wide_pointer_kind};
 use crate::llvm;
 use crate::llvm::debuginfo::{
-    DIBasicType, DIBuilder, DICompositeType, DIDescriptor, DIFile, DIFlags, DILexicalBlock,
-    DIScope, DIType, DebugEmissionKind, DebugNameTableKind,
+    DIBuilder, DICompositeType, DIDescriptor, DIFile, DIFlags, DILexicalBlock, DIScope, DIType,
+    DebugEmissionKind, DebugNameTableKind,
 };
 use crate::value::Value;
 
@@ -491,22 +491,26 @@ pub(crate) fn type_di_node<'ll, 'tcx>(cx: &CodegenCx<'ll, 'tcx>, t: Ty<'tcx>) ->
 // FIXME(mw): Cache this via a regular UniqueTypeId instead of an extra field in the debug context.
 fn recursion_marker_type_di_node<'ll, 'tcx>(cx: &CodegenCx<'ll, 'tcx>) -> &'ll DIType {
     *debug_context(cx).recursion_marker_type.get_or_init(move || {
-        // The choice of type here is pretty arbitrary -
-        // anything reading the debuginfo for a recursive
-        // type is going to see *something* weird - the only
-        // question is what exactly it will see.
-        //
-        // FIXME: the name `<recur_type>` does not fit the naming scheme
-        //        of other types.
-        //
-        // FIXME: it might make sense to use an actual pointer type here
-        //        so that debuggers can show the address.
-        create_basic_type(
-            cx,
-            "<recur_type>",
-            cx.tcx.data_layout.pointer_size,
-            dwarf_const::DW_ATE_unsigned,
-        )
+        unsafe {
+            // The choice of type here is pretty arbitrary -
+            // anything reading the debuginfo for a recursive
+            // type is going to see *something* weird - the only
+            // question is what exactly it will see.
+            //
+            // FIXME: the name `<recur_type>` does not fit the naming scheme
+            //        of other types.
+            //
+            // FIXME: it might make sense to use an actual pointer type here
+            //        so that debuggers can show the address.
+            let name = "<recur_type>";
+            llvm::LLVMRustDIBuilderCreateBasicType(
+                DIB(cx),
+                name.as_c_char_ptr(),
+                name.len(),
+                cx.tcx.data_layout.pointer_size.bits(),
+                dwarf_const::DW_ATE_unsigned,
+            )
+        }
     })
 }
 
@@ -784,7 +788,15 @@ fn build_basic_type_di_node<'ll, 'tcx>(
         _ => bug!("debuginfo::build_basic_type_di_node - `t` is invalid type"),
     };
 
-    let ty_di_node = create_basic_type(cx, name, cx.size_of(t), encoding);
+    let ty_di_node = unsafe {
+        llvm::LLVMRustDIBuilderCreateBasicType(
+            DIB(cx),
+            name.as_c_char_ptr(),
+            name.len(),
+            cx.size_of(t).bits(),
+            encoding,
+        )
+    };
 
     if !cpp_like_debuginfo {
         return DINodeCreationResult::new(ty_di_node, false);
@@ -812,23 +824,6 @@ fn build_basic_type_di_node<'ll, 'tcx>(
     DINodeCreationResult::new(typedef_di_node, false)
 }
 
-fn create_basic_type<'ll, 'tcx>(
-    cx: &CodegenCx<'ll, 'tcx>,
-    name: &str,
-    size: Size,
-    encoding: u32,
-) -> &'ll DIBasicType {
-    unsafe {
-        llvm::LLVMRustDIBuilderCreateBasicType(
-            DIB(cx),
-            name.as_c_char_ptr(),
-            name.len(),
-            size.bits(),
-            encoding,
-        )
-    }
-}
-
 fn build_foreign_type_di_node<'ll, 'tcx>(
     cx: &CodegenCx<'ll, 'tcx>,
     t: Ty<'tcx>,