Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit bf15d59

Browse files
committed
Remove the temporary typedef for LLVMRustDIFlags
If we're already churning all of the debuginfo bindings, we might as well fix this at the same time.
1 parent 63a039a commit bf15d59

File tree

1 file changed

+10
-16
lines changed

1 file changed

+10
-16
lines changed

compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -675,12 +675,6 @@ template <typename DIT> DIT *unwrapDIPtr(LLVMMetadataRef Ref) {
675675
#define DIArray DINodeArray
676676
#define unwrapDI unwrapDIPtr
677677

678-
// FIXME(Zalathar): This is a temporary typedef to avoid churning dozens of
679-
// bindings that are going to be deleted and replaced with their LLVM-C
680-
// equivalents, as part of #134009. After that happens, the remaining bindings
681-
// can be adjusted to use `LLVMDIFlags` instead of relying on this typedef.
682-
typedef LLVMDIFlags LLVMRustDIFlags;
683-
684678
// Statically assert that `LLVMDIFlags` (C) and `DIFlags` (C++) have the same
685679
// layout, at least for the flags we know about. This isn't guaranteed, but is
686680
// likely to remain true, and as long as it is true it makes conversions easy.
@@ -1048,7 +1042,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateFunction(
10481042
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
10491043
size_t NameLen, const char *LinkageName, size_t LinkageNameLen,
10501044
LLVMMetadataRef File, unsigned LineNo, LLVMMetadataRef Ty,
1051-
unsigned ScopeLine, LLVMRustDIFlags Flags, LLVMRustDISPFlags SPFlags,
1045+
unsigned ScopeLine, LLVMDIFlags Flags, LLVMRustDISPFlags SPFlags,
10521046
LLVMValueRef MaybeFn, LLVMMetadataRef TParam, LLVMMetadataRef Decl) {
10531047
DITemplateParameterArray TParams =
10541048
DITemplateParameterArray(unwrap<MDTuple>(TParam));
@@ -1068,7 +1062,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateMethod(
10681062
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
10691063
size_t NameLen, const char *LinkageName, size_t LinkageNameLen,
10701064
LLVMMetadataRef File, unsigned LineNo, LLVMMetadataRef Ty,
1071-
LLVMRustDIFlags Flags, LLVMRustDISPFlags SPFlags, LLVMMetadataRef TParam) {
1065+
LLVMDIFlags Flags, LLVMRustDISPFlags SPFlags, LLVMMetadataRef TParam) {
10721066
DITemplateParameterArray TParams =
10731067
DITemplateParameterArray(unwrap<MDTuple>(TParam));
10741068
DISubprogram::DISPFlags llvmSPFlags = fromRust(SPFlags);
@@ -1112,7 +1106,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreatePointerType(
11121106
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateStructType(
11131107
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
11141108
size_t NameLen, LLVMMetadataRef File, unsigned LineNumber,
1115-
uint64_t SizeInBits, uint32_t AlignInBits, LLVMRustDIFlags Flags,
1109+
uint64_t SizeInBits, uint32_t AlignInBits, LLVMDIFlags Flags,
11161110
LLVMMetadataRef DerivedFrom, LLVMMetadataRef Elements, unsigned RunTimeLang,
11171111
LLVMMetadataRef VTableHolder, const char *UniqueId, size_t UniqueIdLen) {
11181112
return wrap(unwrap(Builder)->createStructType(
@@ -1126,7 +1120,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateStructType(
11261120
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateVariantPart(
11271121
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
11281122
size_t NameLen, LLVMMetadataRef File, unsigned LineNumber,
1129-
uint64_t SizeInBits, uint32_t AlignInBits, LLVMRustDIFlags Flags,
1123+
uint64_t SizeInBits, uint32_t AlignInBits, LLVMDIFlags Flags,
11301124
LLVMMetadataRef Discriminator, LLVMMetadataRef Elements,
11311125
const char *UniqueId, size_t UniqueIdLen) {
11321126
return wrap(unwrap(Builder)->createVariantPart(
@@ -1140,7 +1134,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateVariantPart(
11401134
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateMemberType(
11411135
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
11421136
size_t NameLen, LLVMMetadataRef File, unsigned LineNo, uint64_t SizeInBits,
1143-
uint32_t AlignInBits, uint64_t OffsetInBits, LLVMRustDIFlags Flags,
1137+
uint32_t AlignInBits, uint64_t OffsetInBits, LLVMDIFlags Flags,
11441138
LLVMMetadataRef Ty) {
11451139
return wrap(unwrap(Builder)->createMemberType(
11461140
unwrapDI<DIDescriptor>(Scope), StringRef(Name, NameLen),
@@ -1152,7 +1146,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateVariantMemberType(
11521146
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
11531147
size_t NameLen, LLVMMetadataRef File, unsigned LineNo, uint64_t SizeInBits,
11541148
uint32_t AlignInBits, uint64_t OffsetInBits, LLVMValueRef Discriminant,
1155-
LLVMRustDIFlags Flags, LLVMMetadataRef Ty) {
1149+
LLVMDIFlags Flags, LLVMMetadataRef Ty) {
11561150
llvm::ConstantInt *D = nullptr;
11571151
if (Discriminant) {
11581152
D = unwrap<llvm::ConstantInt>(Discriminant);
@@ -1166,7 +1160,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateVariantMemberType(
11661160
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateStaticMemberType(
11671161
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
11681162
size_t NameLen, LLVMMetadataRef File, unsigned LineNo, LLVMMetadataRef Ty,
1169-
LLVMRustDIFlags Flags, LLVMValueRef val, uint32_t AlignInBits) {
1163+
LLVMDIFlags Flags, LLVMValueRef val, uint32_t AlignInBits) {
11701164
return wrap(unwrap(Builder)->createStaticMemberType(
11711165
unwrapDI<DIDescriptor>(Scope), StringRef(Name, NameLen),
11721166
unwrapDI<DIFile>(File), LineNo, unwrapDI<DIType>(Ty), fromRust(Flags),
@@ -1228,8 +1222,8 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateStaticVariable(
12281222
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateVariable(
12291223
LLVMDIBuilderRef Builder, unsigned Tag, LLVMMetadataRef Scope,
12301224
const char *Name, size_t NameLen, LLVMMetadataRef File, unsigned LineNo,
1231-
LLVMMetadataRef Ty, bool AlwaysPreserve, LLVMRustDIFlags Flags,
1232-
unsigned ArgNo, uint32_t AlignInBits) {
1225+
LLVMMetadataRef Ty, bool AlwaysPreserve, LLVMDIFlags Flags, unsigned ArgNo,
1226+
uint32_t AlignInBits) {
12331227
if (Tag == 0x100) { // DW_TAG_auto_variable
12341228
return wrap(unwrap(Builder)->createAutoVariable(
12351229
unwrapDI<DIDescriptor>(Scope), StringRef(Name, NameLen),
@@ -1303,7 +1297,7 @@ extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateEnumerationType(
13031297
extern "C" LLVMMetadataRef LLVMRustDIBuilderCreateUnionType(
13041298
LLVMDIBuilderRef Builder, LLVMMetadataRef Scope, const char *Name,
13051299
size_t NameLen, LLVMMetadataRef File, unsigned LineNumber,
1306-
uint64_t SizeInBits, uint32_t AlignInBits, LLVMRustDIFlags Flags,
1300+
uint64_t SizeInBits, uint32_t AlignInBits, LLVMDIFlags Flags,
13071301
LLVMMetadataRef Elements, unsigned RunTimeLang, const char *UniqueId,
13081302
size_t UniqueIdLen) {
13091303
return wrap(unwrap(Builder)->createUnionType(

0 commit comments

Comments
 (0)