Skip to content

Commit

Permalink
Auto merge of rust-lang#117948 - aeubanks:dibuilder, r=durin42
Browse files Browse the repository at this point in the history
llvm-wrapper: Pass newly added param to DIBuilder::createStaticMemberType()

This was added in llvm/llvm-project#72234.
DW_TAG_member was the implicit default before.

The LLVM change is quite sinister since due to weakly typed ints and default params, this was still successfully compiling against LLVM but was passing the wrong parameters.
  • Loading branch information
bors committed Nov 16, 2023
2 parents 9144d51 + e2c3e94 commit 6faa181
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -983,6 +983,9 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateStaticMemberType(
unwrapDI<DIType>(Ty),
fromRust(Flags),
unwrap<llvm::ConstantInt>(val),
#if LLVM_VERSION_GE(18, 0)
llvm::dwarf::DW_TAG_member,
#endif
AlignInBits
));
}
Expand Down

0 comments on commit 6faa181

Please sign in to comment.