jimingham wrote: They really do seem odd in the general CompilerType...
Jim > On Dec 5, 2023, at 4:35 PM, cmtice ***@***.***> wrote: > > > @cmtice commented on this pull request. > > In lldb/include/lldb/Symbol/CompilerType.h > <https://github.com/llvm/llvm-project/pull/73472#discussion_r1416471222>: > > > + > + bool IsContextuallyConvertibleToBool() const; > + > + bool IsBasicType() const; > + > + std::string TypeDescription(); > + > + bool CompareTypes(CompilerType rhs) const; > + > + const char *GetTypeTag(); > + > + uint32_t GetNumberOfNonEmptyBaseClasses(); > + > + CompilerType GetTemplateArgumentType(uint32_t idx); > + > + CompilerType GetSmartPtrPointeeType(); > Clarification: I meant "I can remove the helper functions related to smart > pointers." > > — > Reply to this email directly, view it on GitHub > <https://github.com/llvm/llvm-project/pull/73472#discussion_r1416471222>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/ADUPVW4JAG5WJJFUNJSL2HLYH64WZAVCNFSM6AAAAAA73I4PWOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONRWGM3TAOJXGY>. > You are receiving this because you are on a team that was mentioned. > https://github.com/llvm/llvm-project/pull/73472 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
