labath added a comment. The idea seems right, but I'm not too familiar with this part of the code. Someone else ought to look at this too.
================ Comment at: include/lldb/Expression/ExpressionVariable.h:235-236 + virtual bool SetCompilerTypeFromPersistentDecl(ConstString type_name, + CompilerType &compiler_type) = 0; + ---------------- It would be better if this would return `Optional`(or `Expected`)`<CompilerType>` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62797/new/ https://reviews.llvm.org/D62797 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
