Skip to content

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

+10-16
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)