[Lldb-commits] [lldb] [LLDB] Add more helper functions to CompilerType class (second try). (PR #73472)

[via lldb-commits]

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