From 6aba383ab52229968809ddca370809b126369dbc Mon Sep 17 00:00:00 2001 From: Zachary Turner Date: Thu, 26 Feb 2015 23:49:23 +0000 Subject: [PATCH] [llvm-pdbdump] Fix dumping of function pointers and basic types. Function pointers were not correctly handled by the dumper, and they would print as "* name". They now print as "int (__cdecl *name)(int arg1, int arg2)" as they should. Also, doubles were being printed as floats. This fixes that bug as well, and adds tests for all builtin types. as well as a test for function pointers. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@230703 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/DebugInfo/PDB/PDBExtras.h | 1 - lib/DebugInfo/PDB/PDBExtras.cpp | 25 --------------- test/DebugInfo/PDB/Inputs/symbolformat.cpp | 21 +++++++++++-- test/DebugInfo/PDB/Inputs/symbolformat.pdb | Bin 143360 -> 143360 bytes test/DebugInfo/PDB/pdbdump-symbol-format.test | 42 +++++++++++++++++--------- tools/llvm-pdbdump/CMakeLists.txt | 1 + tools/llvm-pdbdump/FunctionDumper.cpp | 16 +++++----- tools/llvm-pdbdump/FunctionDumper.h | 4 +-- tools/llvm-pdbdump/TypeDumper.cpp | 2 +- tools/llvm-pdbdump/TypedefDumper.cpp | 12 +++----- tools/llvm-pdbdump/VariableDumper.cpp | 32 +++++++++++++++++--- tools/llvm-pdbdump/VariableDumper.h | 2 ++ 12 files changed, 93 insertions(+), 65 deletions(-) diff --git a/include/llvm/DebugInfo/PDB/PDBExtras.h b/include/llvm/DebugInfo/PDB/PDBExtras.h index cbbe1716ee0..104fb6f0796 100644 --- a/include/llvm/DebugInfo/PDB/PDBExtras.h +++ b/include/llvm/DebugInfo/PDB/PDBExtras.h @@ -26,7 +26,6 @@ raw_ostream &operator<<(raw_ostream &OS, const PDB_ThunkOrdinal &Thunk); raw_ostream &operator<<(raw_ostream &OS, const PDB_Checksum &Checksum); raw_ostream &operator<<(raw_ostream &OS, const PDB_Lang &Lang); raw_ostream &operator<<(raw_ostream &OS, const PDB_SymType &Tag); -raw_ostream &operator<<(raw_ostream &OS, const PDB_BuiltinType &Type); raw_ostream &operator<<(raw_ostream &OS, const PDB_UniqueId &Id); raw_ostream &operator<<(raw_ostream &OS, const Variant &Value); diff --git a/lib/DebugInfo/PDB/PDBExtras.cpp b/lib/DebugInfo/PDB/PDBExtras.cpp index 1002b2e2dad..5a2856ff254 100644 --- a/lib/DebugInfo/PDB/PDBExtras.cpp +++ b/lib/DebugInfo/PDB/PDBExtras.cpp @@ -247,31 +247,6 @@ raw_ostream &llvm::operator<<(raw_ostream &OS, const PDB_SymType &Tag) { return OS; } -raw_ostream &llvm::operator<<(raw_ostream &OS, const PDB_BuiltinType &Type) { - switch (Type) { - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Void, "void", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Char, "char", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, WCharT, "wchar_t", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Int, "int", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, UInt, "uint", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Float, "float", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, BCD, "BCD", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Bool, "bool", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Long, "long", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, ULong, "ulong", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Currency, "CURRENCY", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Date, "DATE", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Variant, "VARIANT", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Complex, "complex", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, Bitfield, "bitfield", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, BSTR, "BSTR", OS) - CASE_OUTPUT_ENUM_CLASS_STR(PDB_BuiltinType, HResult, "HRESULT", OS) - default: - break; - } - return OS; -} - raw_ostream &llvm::operator<<(raw_ostream &OS, const PDB_UniqueId &Id) { static const char *Lookup = "0123456789ABCDEF"; diff --git a/test/DebugInfo/PDB/Inputs/symbolformat.cpp b/test/DebugInfo/PDB/Inputs/symbolformat.cpp index c069a35c7b9..237913e7221 100644 --- a/test/DebugInfo/PDB/Inputs/symbolformat.cpp +++ b/test/DebugInfo/PDB/Inputs/symbolformat.cpp @@ -26,22 +26,39 @@ struct VirtualBase { struct B : public A, protected virtual VirtualBase { void PureFunc() override {} +}; +struct MemberTest { enum NestedEnum { FirstVal, SecondVal }; typedef int NestedTypedef; - NestedEnum EnumVar; - NestedTypedef TypedefVar; + + NestedEnum m_nested_enum; + NestedTypedef m_typedef; + bool m_bool; + char m_char; + wchar_t m_wchar_t; + int m_int; + unsigned m_unsigned; + long m_long; + unsigned long m_unsigned_long; + __int64 m_int64; + unsigned __int64 m_unsigned_int64; + float m_float; + double m_double; + void (*m_pfn_2_args)(int, double); }; typedef int IntType; typedef A ClassAType; int main(int argc, char **argv) { + // Force symbol references so the linker generates debug info B b; + MemberTest members; auto PureAddr = &B::PureFunc; auto VirtualAddr = &A::PureFunc; auto RegularAddr = &A::RegularFunc; diff --git a/test/DebugInfo/PDB/Inputs/symbolformat.pdb b/test/DebugInfo/PDB/Inputs/symbolformat.pdb index 183870aa164fd1dc971159b602cca4b8ceefbd49..28832d4090fab93e9c9b48e4db52391aceb5013b 100644 GIT binary patch delta 6186 zcmeHL4Nz3q6+ZVZtGf&AE`LG*WfjYx$^dB<9TRp@n#4$51rxO*Ke7S|Kg*wDl(IOI zu``*@!fo^H*yz%k#7P=k>4G>-+JZXmw3*3BQ=K$Trr2s{I&mgwZEI4S?RVZj6s==_ z(sVjWE_{3LJ@?#m-#h2L^Uk}zNt18Vo3h$tQK4=pL09kiEx!=xDO!5I5{HtDci zjVIs!16`jUMh)aeB<+9cy>ebuHnFY#{P%O#4z%=+=PX;%6->>$XzPh(RlrVQe+tp> zBZ$@-F+e+rWvK?DdR~s!1vmte8)yd3KyEMQZlLeQToM3riUU}$4M+f@04op;L;`kT z77)WCv(d2tbAUt|{hCjFTM?UO{;Ig_Itcy4jbM+U9QQ=U|*r35&Oaiw?R4aSOR>+5(w#e??VT*vuYUe5ukk30Gine0!tB}hB z0`h(5@?fB9mvz7!C2HmPnF3kljk{xYy7YM+GT?T~NpF(8?zT%)YlnDDcDE)j zc^n#7HCDH`yV#(SF;k&XXft>v&7DmeA3IsWp9H^7j<;qeuswMe6R`0t{H=|-PyVen z&9)0Oe4~PdYsCteEZUGQ{bhDp+m>pq#+*a;wUsFqLcZ0uMR?^G+Mg6YIoRGJ{Bpc( zPmdR?8EmQ@05^7it=1!2LFD5gSP9BIJqWrCvPL5o4lKv#io0bLDR z1qvBo4d@p^uYqz9B53B3U936RzU`}5&%NMZUGe(Fg+DI{TiXEvue?y65}k@AY=7qX zz{$j=gP$9ZVHSA%aqZs%jss)BS>PgY9f(qhvVez`peT9J@|nZP39EfXhwXgjFQ-Lt z=~z|Yf}g&5{~L!tPKWt&4g56b`y6=EddC*ZoxW1-nOiD9@L98c@G&AEd2BJBWuum& zym#<8zd2-nZ`=bAyiNK(%mnV>%>%Crn5i`#W54W$hVY|uYriF!sn9hf)6y8hSrpHS zimP!zQErE)s{=C0v@AxI6TRsPlZZwq{&U-Et2?U0Gj-CuFJ08hCHqoDy?kU}nm80} z*r!Ghe2VY>NVHI;*`FyUWj=ln$<6*W&fN!W0|EPnY*-Oyr{^@=6Fs7j$Mx}qPKR{b zr_)P1osYHX();>2T9NLF!hUyb@a4c(8xp_y_!^7IAZrX1!}qNt^DKtxp-+|$&x$4=}caJ9{xAqUG3x^aZQlx1po>T6;p*1pZ#AJ0theAa~m4(Gl=Q3F{RA98y?3j94 z8cY*=WmE8wbRJJdT5BE2M0PG6&Q$)XQsEBvo!Da@a$i&i&Mw|^Nxz}zb=s}dxxK0~%0|=ew>$Fs-2?B;$Bqs+g!B@^26*>Id#d-CZ^C7^!9OM932mzo9f$s

tmeR8S}s?60~QH85tfVIAIF^vrDP-i^vYCXUT5E!Qq&El+8UI|2M~!Qmix6*F(d z<}@?q8f7Znp-rGHyXh_VeYk0FaxG7+SQX~5L>0hw6Kmt~;=!})K+vpIg*u8`AtrdP z{_UCC@Mbr*)I5O-A^eUv%A@1y5jN^l`dxE`ke9~qpI3!dchk<=;Kbxp$%u(}P9@G~ z2B!+%(P6i1moobFS*gk4$1B&LN`d$J*U!K40Z89z203+Vt8mNG)0`<9@Ox7BonC}T zW$1LAmRro)#s-)@6k^qfOi-_8iQL6s#=OPH z^{TM5tUys}A;n&SeomMl%NE#%vFghf@uE_FAVVan$FfD7neW+v>1wq#L&W88$B>nj z1CdZw3Ce8)8Dc5kK;!m6;TOgx^~Efa7Oh~H0y1)Wt~98-bHpMwD_bl~F4ja% zn&^{IQ|f7|@wAnD+B?*WY%#a^));!NBx2d?tk(rt$z*2_a#Cu-Vzx~)Qy*+146$}> z*QmaO&3YQa4p*L!Rkz_zq7w|7ng}Z2?e)}pb};*oVSXMYP1ICb)6&v7&GUk125)=) zExA_kEa2^8W@QIW>x=@gp*cM6gp8enByYOdJRryaE84hf(28r+n#j_)9v-q z=w_!lYRixOdA`t+J(ec}@kV|(t3o&a3LG;~MT(#UM-=KoP50q? zK_`Dr^qXCR#s(Ey_mqKtc39A>-!xF%4-E7us`0rcf{xBoXxl-e^GgJE3B17k=EeYq^cmToS1y1I_u>5nc6<>bruTv&3LgQX%n4;^OzpfX@gFg$LQ0?S9FR<*ZBE5 zt=4I`PD^BUVDZiReJdXl=3(lWpr~GGH86$#%fNnsTNQYFfrEe#HQT>c=l`G8j7X)Q z>p_m0#8YX%BMolYv1eK4g~J JUT`S(zXKpS4r2fS delta 7921 zcmeHM3viUx6+ZXhY_hv~?Izg-@>~c>kc7AajFAD{4WtSNSss!EdF+OShlXUcNfd#Y z*lGtmRkFF@W~{AXg?)d1~Wr7mG0hJ#2d|{VQx^Ie|U?v&?*Upe<-4(qq z=b#WCSu{V5~)XzS7n>3sNQ^2^bmfaYjV9K>*qf+y!)} z66NTL_C_Ls(4`Qi>xjz1<8U5fE2s@~QxCX+mpKpRxxlW(->fe3Rs0lX-HuZnIW5ORJj(dOPa&ll-=CktwKU ze@yaiU#TdN8BHmIy#9@4{OGUjV-a z&WIl8i>5Z#p|BFpLBB>YRK)_HCp8AsGeMg@tm7q0cr(g#ldPPqgwW$LO2~?bp{Pp8 z3OP;*G|6+FB1zDA9) z+J}@Q9~2Ez0qFIh(?REf7J`<6UJF_dY6o2oS_HZZbOxv!bS7vu=q%7vpcj>=Ew$oM z>pJBJ!w>3zz2;YY{9X560rpUWG7HdoRQAeW1tWE@*)B@~vJ(;A_pQvX`6_iE(#JL9%!ydno z&nC$}u1+R+j&Zzf@NDuvv=Ex=i!ERp>0^LfO{HRsV0(ypo`U{uMDD`o+O5sVPo?WE zQ{;d@+j<1*>%gzjSL12+gmP9%V`sKlC8u|$iPdspXO`&d-TcJkv0{oe9?TO*WD)+m zWYxhexz)SrKWjU!kS1*Z{@ZM46B`~?_gq*5FMx&pTpNnzYG)gY6HT?DI2BYI$~wyS z4fa&Ep~;{*NaOSvw+(*$!F$)Wo^zybJXZeZfdmr@!CpqP65t@=Z2;3eW*el%;>f(f z37JzIoL68@JA>wBhp;JFnEUX@^thI;(&%Q5E)1u&a;;i=66$b0zm|SdCtb%&{H}!F zXM(Ha;fNUf7nvxR;w;ctDF2%N%$Xvb4DGWUwWIbME%oaiPKRTuv#hd`ZH7qIsvRVG zpg$Fc>$U!D1&8Ec`wQ8aWezNo_YV|gBv20K9Y-$)qL6y>VgW+DBqN} zZRvGeJe%vWO+)gA2C0vuD2rDKQ}IbE=?e@Q4ep4@yCBa|S6^Bm(^D~c>^sJ=t>k@;MX zl>3LXbqTap9%@MuCG!2@nVAPsjomk!In=zbxc3;6YI9h*`Vm&SLnCQebpE%g|EfHZ zYL7yuqDW9K9Z8X=Mk=uYi%+uQ-HiWdd<&+s!VS)Qkz>PR^g_;jrr)f;y1CqM-Oq=oUC{)GhFGo~K3gH&o-qw-QI_f>fpNhbH}N z{%8<2+P-psl>JX$4)#AWajO5x{wVvO?2p<(%Rq}j*&k*9ll@WlKlyB#1?mR94zwDy z81xkAMSrv}^_6$t>HF~J&lLCE!_{wfJal#ZQF&|0MEPNHS+A{l(D=Mrd}4)*y&NLx zaq~8jFmqg}am8GL__TJupNItBtd)U*T#+q1E7(s`)yx z7V!peth*)6mF)Z+U z#77HU6KI_#QbcWF&oq%gcLQ=+F24S-TwF6O*Ji{y?E+Y?T2K!4kjn^ntUB=NG;vGO zCa|5_@b5r;^zbuD5216FJGH9nzyqEfkrar{71xUTz}#Gs7dwrjFb||ObL)YDJdqU` zLK!hbI!mm9ZMmW_stGMXyg=Smk+X1yTHc_RZ$Sp!l3Qz=nrmzL=Et*AG3#R5!80^6 zJZ4!@CmJVERp8ILB0uMDBzY3}o_FuKMH5L1`M_@uc=AM=-fYEr8h9vQGRRVP1U}6Z`4fM1g`MYbyi0A2#a-#~LAqG0j(GNB zXvIC=+M3!8%+~bla)UHskZd|R@F2|>2ifEFNBqxR^^&?U0S+u^p|7^CVLeX=QJ^tf zB=+r^A?_J>2kfhIi2d=^>_(&-uN|&LSvz$2YTgDu5VEVZ>k3{UQ^vkN7UETsW#Jc= zDKXHN;!%to`LNUEP9F(D;mL6M^`;{42xtee3E;p_2OmLQA&k z=si4gM6FRM@kJfo5<~R#F&*W-qR_r+3LUA|(X;TXKS)!k3a(-OK9tRc72PRl5z0DF z2&&woBj;*`GRkm|ykF3{zbSO?SslH}m@cwu*K<008ovJT?iMr;FDMqPP&?WmiNUZv zH}^Or?X9-)i-GE^uotU?rbIl9Ks)3 zyRZ(G5);b+ - 0x-] ; SYM_FORMAT: symbolformat-fpo.obj -; SYM_FORMAT-DAG: func [0x001130+0 - 0x001137-1] (FPO) uint32_t __cdecl fpo_func(uint32_t n) +; SYM_FORMAT-DAG: func [{{.*}}] (FPO) unsigned __cdecl fpo_func(unsigned n) ; SYM_FORMAT: symbolformat.obj -; SYM_FORMAT-DAG: func [0x001140+3 - 0x001147-2] (EBP) int32_t __cdecl _purecall() -; SYM_FORMAT-DAG: func [0x001150+6 - 0x0011b6-4] (EBP) int32_t __cdecl main(int32_t argc, char** argv) -; SYM_FORMAT-DAG: func [0x0010b0+7 - 0x0010c7-4] (EBP) void A::A() -; SYM_FORMAT-DAG: func [0x0011c0+7 - 0x0011f1-6] (EBP) void B::B() -; SYM_FORMAT-DAG: thunk [0x000010f6 - 0x000010fa] (Pcode) B::`vcall'{0}' -; SYM_FORMAT-DAG: func [0x001100+7 - 0x00110b-4] (EBP) virtual void B::PureFunc() -; SYM_FORMAT-DAG: func [0x001110+7 - 0x00111b-4] (EBP) void A::RegularFunc() -; SYM_FORMAT-DAG: func [0x001120+7 - 0x00112b-4] (EBP) virtual void A::VirtualFunc() +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) int __cdecl _purecall() +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) int __cdecl main(int argc, char** argv) +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) void A::A() +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) void B::B() +; SYM_FORMAT-DAG: thunk [{{.*}}] (Pcode) B::`vcall'{0}' +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) virtual void B::PureFunc() +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) void A::RegularFunc() +; SYM_FORMAT-DAG: func [{{.*}}] (EBP) virtual void A::VirtualFunc() ; TYPES_FORMAT: Enums ; TYPES_FORMAT-DAG: enum TestEnum ; TYPES_FORMAT-DAG: enum TestEnumClass ; TYPES_FORMAT: Function Signatures -; TYPES_FORMAT-DAG: int32_t __cdecl () -; TYPES_FORMAT-DAG: int32_t __cdecl (int32_t, char**) +; TYPES_FORMAT-DAG: int __cdecl () +; TYPES_FORMAT-DAG: int __cdecl (int, char**) ; TYPES_FORMAT-DAG: void (A::)() ; TYPES_FORMAT-DAG: void (B::)() ; TYPES_FORMAT-DAG: void (B::)(B&) @@ -30,7 +30,7 @@ ; TYPES_FORMAT-DAG: void (A::)() ; TYPES_FORMAT-DAG: A& (A::)(A&) ; TYPES_FORMAT: Typedefs -; TYPES_FORMAT-DAG: typedef int32_t IntType +; TYPES_FORMAT-DAG: typedef int IntType ; TYPES_FORMAT-DAG: typedef class A ClassAType ; TYPES_FORMAT: Classes ; TYPES_FORMAT-DAG: class A @@ -43,7 +43,19 @@ ; FULL_CLASS: virtual void VirtualFunc() ; FULL_CLASS: void RegularFunc() ; FULL_CLASS: } -; FULL_CLASS-DAG: class B { -; FULL_CLASS: public: -; FULL_CLASS: virtual void PureFunc() +; FULL_CLASS-DAG: class MemberTest { +; FULL_CLASS: data +0x00 MemberTest::NestedEnum m_nested_enum +; FULL_CLASS: data +0x04 int m_typedef +; FULL_CLASS: data +0x08 bool m_bool +; FULL_CLASS: data +0x09 char m_char +; FULL_CLASS: data +0x0a wchar_t m_wchar_t +; FULL_CLASS: data +0x0c int m_int +; FULL_CLASS: data +0x10 unsigned m_unsigned +; FULL_CLASS: data +0x14 long m_long +; FULL_CLASS: data +0x18 unsigned long m_unsigned_long +; FULL_CLASS: data +0x20 __int64 m_int64 +; FULL_CLASS: data +0x28 unsigned __int64 m_unsigned_int64 +; FULL_CLASS: data +0x30 float m_float +; FULL_CLASS: data +0x38 double m_double +; FULL_CLASS: data +0x40 void (__cdecl *m_pfn_2_args)(int, double) ; FULL_CLASS: } \ No newline at end of file diff --git a/tools/llvm-pdbdump/CMakeLists.txt b/tools/llvm-pdbdump/CMakeLists.txt index 0519bf0634c..6dae9147bc2 100644 --- a/tools/llvm-pdbdump/CMakeLists.txt +++ b/tools/llvm-pdbdump/CMakeLists.txt @@ -5,6 +5,7 @@ set(LLVM_LINK_COMPONENTS add_llvm_tool(llvm-pdbdump llvm-pdbdump.cpp + BuiltinDumper.cpp ClassDefinitionDumper.cpp CompilandDumper.cpp FunctionDumper.cpp diff --git a/tools/llvm-pdbdump/FunctionDumper.cpp b/tools/llvm-pdbdump/FunctionDumper.cpp index e65983034a6..1a1defed042 100644 --- a/tools/llvm-pdbdump/FunctionDumper.cpp +++ b/tools/llvm-pdbdump/FunctionDumper.cpp @@ -8,6 +8,7 @@ //===----------------------------------------------------------------------===// #include "FunctionDumper.h" +#include "BuiltinDumper.h" #include "llvm-pdbdump.h" #include "llvm/DebugInfo/PDB/IPDBSession.h" @@ -16,7 +17,6 @@ #include "llvm/DebugInfo/PDB/PDBSymbolFuncDebugEnd.h" #include "llvm/DebugInfo/PDB/PDBSymbolFuncDebugStart.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeArray.h" -#include "llvm/DebugInfo/PDB/PDBSymbolTypeBuiltin.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeEnum.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeFunctionArg.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeFunctionSig.h" @@ -45,7 +45,8 @@ void dumpClassParentWithScopeOperator(const T &Symbol, llvm::raw_ostream &OS, FunctionDumper::FunctionDumper() : PDBSymDumper(true) {} void FunctionDumper::start(const PDBSymbolTypeFunctionSig &Symbol, - PointerType Pointer, raw_ostream &OS) { + const char *Name, PointerType Pointer, + raw_ostream &OS) { auto ReturnType = Symbol.getReturnType(); ReturnType->dump(OS, 0, *this); OS << " "; @@ -70,13 +71,14 @@ void FunctionDumper::start(const PDBSymbolTypeFunctionSig &Symbol, OS << "("; if (ShouldDumpCallingConvention) OS << CC << " "; - OS << Symbol.getCallingConvention() << " "; if (ClassParent) OS << ClassParent->getName() << "::"; if (Pointer == PointerType::Reference) OS << "&"; else OS << "*"; + if (Name) + OS << Name; OS << ")"; } @@ -185,10 +187,8 @@ void FunctionDumper::dump(const PDBSymbolTypeArray &Symbol, raw_ostream &OS, void FunctionDumper::dump(const PDBSymbolTypeBuiltin &Symbol, raw_ostream &OS, int Indent) { - PDB_BuiltinType Type = Symbol.getBuiltinType(); - OS << Type; - if (Type == PDB_BuiltinType::UInt || Type == PDB_BuiltinType::Int) - OS << (8 * Symbol.getLength()) << "_t"; + BuiltinDumper Dumper; + Dumper.start(Symbol, OS); } void FunctionDumper::dump(const PDBSymbolTypeEnum &Symbol, raw_ostream &OS, @@ -226,7 +226,7 @@ void FunctionDumper::dump(const PDBSymbolTypePointer &Symbol, raw_ostream &OS, FunctionDumper NestedDumper; PointerType Pointer = Symbol.isReference() ? PointerType::Reference : PointerType::Pointer; - NestedDumper.start(*FuncSig, Pointer, OS); + NestedDumper.start(*FuncSig, nullptr, Pointer, OS); } else { if (Symbol.isConstType()) OS << "const "; diff --git a/tools/llvm-pdbdump/FunctionDumper.h b/tools/llvm-pdbdump/FunctionDumper.h index f9338cb8764..63aa5658f60 100644 --- a/tools/llvm-pdbdump/FunctionDumper.h +++ b/tools/llvm-pdbdump/FunctionDumper.h @@ -20,8 +20,8 @@ public: enum class PointerType { None, Pointer, Reference }; - void start(const PDBSymbolTypeFunctionSig &Symbol, PointerType Pointer, - raw_ostream &OS); + void start(const PDBSymbolTypeFunctionSig &Symbol, const char *Name, + PointerType Pointer, raw_ostream &OS); void start(const PDBSymbolFunc &Symbol, PointerType Pointer, raw_ostream &OS, int Indent); diff --git a/tools/llvm-pdbdump/TypeDumper.cpp b/tools/llvm-pdbdump/TypeDumper.cpp index 3131e9f00cc..7b705961441 100644 --- a/tools/llvm-pdbdump/TypeDumper.cpp +++ b/tools/llvm-pdbdump/TypeDumper.cpp @@ -68,7 +68,7 @@ void TypeDumper::dump(const PDBSymbolTypeFunctionSig &Symbol, raw_ostream &OS, OS << newline(Indent); FunctionDumper Dumper; - Dumper.start(Symbol, FunctionDumper::PointerType::None, OS); + Dumper.start(Symbol, nullptr, FunctionDumper::PointerType::None, OS); } void TypeDumper::dump(const PDBSymbolTypeTypedef &Symbol, raw_ostream &OS, diff --git a/tools/llvm-pdbdump/TypedefDumper.cpp b/tools/llvm-pdbdump/TypedefDumper.cpp index 6eea6b69d95..7bb4b9d2d43 100644 --- a/tools/llvm-pdbdump/TypedefDumper.cpp +++ b/tools/llvm-pdbdump/TypedefDumper.cpp @@ -9,12 +9,12 @@ #include "TypedefDumper.h" +#include "BuiltinDumper.h" #include "FunctionDumper.h" #include "llvm-pdbdump.h" #include "llvm/DebugInfo/PDB/IPDBSession.h" #include "llvm/DebugInfo/PDB/PDBExtras.h" -#include "llvm/DebugInfo/PDB/PDBSymbolTypeBuiltin.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeEnum.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeFunctionSig.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypePointer.h" @@ -39,10 +39,8 @@ void TypedefDumper::dump(const PDBSymbolTypeArray &Symbol, raw_ostream &OS, void TypedefDumper::dump(const PDBSymbolTypeBuiltin &Symbol, raw_ostream &OS, int Indent) { - PDB_BuiltinType Type = Symbol.getBuiltinType(); - OS << Type; - if (Type == PDB_BuiltinType::UInt || Type == PDB_BuiltinType::Int) - OS << (8 * Symbol.getLength()) << "_t"; + BuiltinDumper Dumper; + Dumper.start(Symbol, OS); } void TypedefDumper::dump(const PDBSymbolTypeEnum &Symbol, raw_ostream &OS, @@ -65,7 +63,7 @@ void TypedefDumper::dump(const PDBSymbolTypePointer &Symbol, raw_ostream &OS, if (Symbol.isReference()) Pointer = FunctionDumper::PointerType::Reference; FunctionDumper NestedDumper; - NestedDumper.start(*FuncSig, Pointer, OS); + NestedDumper.start(*FuncSig, nullptr, Pointer, OS); } else { PointeeType->dump(OS, Indent, *this); OS << ((Symbol.isReference()) ? "&" : "*"); @@ -75,7 +73,7 @@ void TypedefDumper::dump(const PDBSymbolTypePointer &Symbol, raw_ostream &OS, void TypedefDumper::dump(const PDBSymbolTypeFunctionSig &Symbol, raw_ostream &OS, int Indent) { FunctionDumper Dumper; - Dumper.start(Symbol, FunctionDumper::PointerType::None, OS); + Dumper.start(Symbol, nullptr, FunctionDumper::PointerType::None, OS); } void TypedefDumper::dump(const PDBSymbolTypeUDT &Symbol, raw_ostream &OS, diff --git a/tools/llvm-pdbdump/VariableDumper.cpp b/tools/llvm-pdbdump/VariableDumper.cpp index 913cfee6622..371c270443a 100644 --- a/tools/llvm-pdbdump/VariableDumper.cpp +++ b/tools/llvm-pdbdump/VariableDumper.cpp @@ -9,13 +9,14 @@ #include "VariableDumper.h" +#include "BuiltinDumper.h" #include "llvm-pdbdump.h" #include "FunctionDumper.h" #include "llvm/DebugInfo/PDB/PDBSymbolData.h" #include "llvm/DebugInfo/PDB/PDBSymbolFunc.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeArray.h" -#include "llvm/DebugInfo/PDB/PDBSymbolTypeBuiltin.h" +#include "llvm/DebugInfo/PDB/PDBSymbolTypeFunctionSig.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypePointer.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeTypedef.h" #include "llvm/DebugInfo/PDB/PDBSymbolTypeEnum.h" @@ -57,7 +58,8 @@ void VariableDumper::start(const PDBSymbolData &Var, raw_ostream &OS, void VariableDumper::dump(const PDBSymbolTypeBuiltin &Symbol, raw_ostream &OS, int Indent) { - OS << Symbol.getBuiltinType(); + BuiltinDumper Dumper; + Dumper.start(Symbol, OS); } void VariableDumper::dump(const PDBSymbolTypeEnum &Symbol, raw_ostream &OS, @@ -114,7 +116,29 @@ void VariableDumper::dumpSymbolTypeAndName(const PDBSymbol &Type, ElementType->dump(OS, 0, *this); OS << " " << Name << IndexStream.str(); } else { - Type.dump(OS, 0, *this); - OS << " " << Name; + if (!tryDumpFunctionPointer(Type, Name, OS)) { + Type.dump(OS, 0, *this); + OS << " " << Name; + } + } +} + +bool VariableDumper::tryDumpFunctionPointer(const PDBSymbol &Type, + StringRef Name, raw_ostream &OS) { + // Function pointers come across as pointers to function signatures. But the + // signature carries no name, so we have to handle this case separately. + if (auto *PointerType = dyn_cast(&Type)) { + auto PointeeType = PointerType->getPointeeType(); + if (auto *FunctionSig = + dyn_cast(PointeeType.get())) { + FunctionDumper Dumper; + FunctionDumper::PointerType PT = FunctionDumper::PointerType::Pointer; + if (PointerType->isReference()) + PT = FunctionDumper::PointerType::Reference; + std::string NameStr(Name.begin(), Name.end()); + Dumper.start(*FunctionSig, NameStr.c_str(), PT, OS); + return true; + } } + return false; } diff --git a/tools/llvm-pdbdump/VariableDumper.h b/tools/llvm-pdbdump/VariableDumper.h index e6e71faca31..e8832e3d633 100644 --- a/tools/llvm-pdbdump/VariableDumper.h +++ b/tools/llvm-pdbdump/VariableDumper.h @@ -37,6 +37,8 @@ public: private: void dumpSymbolTypeAndName(const PDBSymbol &Type, StringRef Name, raw_ostream &OS); + bool tryDumpFunctionPointer(const PDBSymbol &Type, StringRef Name, + raw_ostream &OS); }; } -- 2.11.0