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

[Alex Langford via Phabricator via lldb-commits]

xiaobai marked an inline comment as done.
xiaobai added inline comments.


================
Comment at: include/lldb/Expression/ExpressionVariable.h:235-236
 
+  virtual bool SetCompilerTypeFromPersistentDecl(ConstString type_name,
+                                                 CompilerType &compiler_type) = 0;
+
----------------
labath wrote:
> It would be better if this would return `Optional`(or `Expected`)`<CompilerType>`
Let me try that out, I'll update this later.


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

https://reviews.llvm.org/D62797