Skip to content

Commit 7d93cc9

Browse files
committed
Add FIXME for auditing optional parameters passed to DIBuilder
1 parent 181b8f9 commit 7d93cc9

File tree

1 file changed

+4
-0
lines changed
  • compiler/rustc_codegen_llvm/src/llvm

1 file changed

+4
-0
lines changed

compiler/rustc_codegen_llvm/src/llvm/ffi.rs

+4
Original file line numberDiff line numberDiff line change
@@ -1626,6 +1626,10 @@ unsafe extern "C" {
16261626

16271627
// FFI bindings for `DIBuilder` functions in the LLVM-C API.
16281628
// Try to keep these in the same order as in `llvm/include/llvm-c/DebugInfo.h`.
1629+
//
1630+
// FIXME(#134001): Audit all `Option` parameters, especially in lists, to check
1631+
// that they really are nullable on the C/C++ side. LLVM doesn't appear to
1632+
// actually document which ones are nullable.
16291633
unsafe extern "C" {
16301634
pub(crate) fn LLVMCreateDIBuilder<'ll>(M: &'ll Module) -> *mut DIBuilder<'ll>;
16311635
pub(crate) fn LLVMDisposeDIBuilder<'ll>(Builder: ptr::NonNull<DIBuilder<'ll>>);

0 commit comments

Comments
 (0)