[Lldb-commits] [PATCH] D63240: [Core] Generalize ValueObject::IsRuntimeSupportValue

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 26 17:13:47 PDT 2019

xiaobai added a comment.

In D63240#1560074 <https://reviews.llvm.org/D63240#1560074>, @jingham wrote:

> I was talking specifically about which runtime should be asked the question.  It seemed to me most straightforward to invert the order in which the question was asked, but another way to get the same effect without rejiggering the API's at this point is to have ValueObject::IsRuntimeSupportValue do:
>   SymbolContextScope *sym_ctx_scope = GetSymbolContextScope();
>   LanguageType languageToAsk = eLanguageTypeUnknown;
>   if (sym_ctx_scope) {
>      Function *func = sym_ctx_scope->CalculateSymbolContextFunction();
>      if (func)
>       languageToAsk = func->GetLanguage();
>     else  //Do something similar for the CU.
> Then get the languageRuntime for languageToAsk and ask it.  That way you don't have to monkey with the runtime language of the ValueObject, but you are still asking the right actor.

Right, I see. I'll try to implement something along the lines of what you're suggesting and report back. Thanks for the suggestion!



More information about the lldb-commits mailing list