From 8b56ca61e1f174d08d78f5aab3c83633c48af3a5 Mon Sep 17 00:00:00 2001 From: Manman Ren Date: Fri, 6 Sep 2013 23:54:23 +0000 Subject: [PATCH] Debug Info: pass in VTableHolder as DIType instead of MDNode *. Remove one cast and improve readability. No functionality change. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@190225 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/DIBuilder.h | 7 ++++--- lib/IR/DIBuilder.cpp | 10 +++++----- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/include/llvm/DIBuilder.h b/include/llvm/DIBuilder.h index d4dfadc5c76..39ff317a748 100644 --- a/include/llvm/DIBuilder.h +++ b/include/llvm/DIBuilder.h @@ -17,6 +17,7 @@ #include "llvm/ADT/ArrayRef.h" #include "llvm/ADT/StringRef.h" +#include "llvm/DebugInfo.h" #include "llvm/Support/DataTypes.h" #include "llvm/Support/ValueHandle.h" @@ -288,7 +289,7 @@ namespace llvm { uint64_t SizeInBits, uint64_t AlignInBits, uint64_t OffsetInBits, unsigned Flags, DIType DerivedFrom, DIArray Elements, - MDNode *VTableHolder = 0, + DIType VTableHolder = NULL, MDNode *TemplateParms = 0, StringRef UniqueIdentifier = StringRef()); @@ -308,7 +309,7 @@ namespace llvm { uint64_t SizeInBits, uint64_t AlignInBits, unsigned Flags, DIType DerivedFrom, DIArray Elements, unsigned RunTimeLang = 0, - MDNode *VTableHolder = 0, + DIType VTableHolder = NULL, StringRef UniqueIdentifier = StringRef()); /// createUnionType - Create debugging information entry for an union. @@ -586,7 +587,7 @@ namespace llvm { DICompositeType Ty, bool isLocalToUnit, bool isDefinition, unsigned Virtuality = 0, unsigned VTableIndex = 0, - MDNode *VTableHolder = 0, + DIType VTableHolder = NULL, unsigned Flags = 0, bool isOptimized = false, Function *Fn = 0, diff --git a/lib/IR/DIBuilder.cpp b/lib/IR/DIBuilder.cpp index 72205f67873..cc06d4ba25c 100644 --- a/lib/IR/DIBuilder.cpp +++ b/lib/IR/DIBuilder.cpp @@ -607,7 +607,7 @@ DICompositeType DIBuilder::createClassType(DIDescriptor Context, StringRef Name, uint64_t OffsetInBits, unsigned Flags, DIType DerivedFrom, DIArray Elements, - MDNode *VTableHolder, + DIType VTableHolder, MDNode *TemplateParams, StringRef UniqueIdentifier) { assert((!Context || Context.isScope() || Context.isType()) && @@ -626,7 +626,7 @@ DICompositeType DIBuilder::createClassType(DIDescriptor Context, StringRef Name, DerivedFrom, Elements, ConstantInt::get(Type::getInt32Ty(VMContext), 0), - DIType(VTableHolder).generateRef(), + VTableHolder.generateRef(), TemplateParams, UniqueIdentifier.empty() ? NULL : MDString::get(VMContext, UniqueIdentifier) }; @@ -647,7 +647,7 @@ DICompositeType DIBuilder::createStructType(DIDescriptor Context, unsigned Flags, DIType DerivedFrom, DIArray Elements, unsigned RunTimeLang, - MDNode *VTableHolder, + DIType VTableHolder, StringRef UniqueIdentifier) { // TAG_structure_type is encoded in DICompositeType format. Value *Elts[] = { @@ -663,7 +663,7 @@ DICompositeType DIBuilder::createStructType(DIDescriptor Context, DerivedFrom, Elements, ConstantInt::get(Type::getInt32Ty(VMContext), RunTimeLang), - DIType(VTableHolder).generateRef(), + VTableHolder.generateRef(), NULL, UniqueIdentifier.empty() ? NULL : MDString::get(VMContext, UniqueIdentifier) }; @@ -1104,7 +1104,7 @@ DISubprogram DIBuilder::createMethod(DIDescriptor Context, bool isLocalToUnit, bool isDefinition, unsigned VK, unsigned VIndex, - MDNode *VTableHolder, + DIType VTableHolder, unsigned Flags, bool isOptimized, Function *Fn, -- 2.34.1