[Lldb-commits] [PATCH] D62797: [Expression] Add PersistentExpressionState::GetCompilerTypeFromPersistentDecl

[Alex Langford via Phabricator via lldb-commits]

xiaobai added a comment.

In D62797#1528483 <https://reviews.llvm.org/D62797#1528483>, @jingham wrote:

> I have no problem with the change in general.  However, you've introduced the possibility of name collision between convenience type definition in various languages.  So it would be good to run through the persistent DECL's for ALL the supported languages and report collisions.  Probably also need to add a -l flag to pick a language?


Hmm, yes that's true. Would you like me to implement that in this patch as well or would you be okay with a follow up patch?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62797/new/

https://reviews.llvm.org/D62797