devnexen added a comment. In D68723#1703379 <https://reviews.llvm.org/D68723#1703379>, @MaskRay wrote:
> In D68723#1703322 <https://reviews.llvm.org/D68723#1703322>, @devnexen wrote: > > > @dim I LGTMed this but would it a real issue for you to take on a less > > disruptive approach proposed by MaskRay ? > > > I think we should do what @labath suggested (changing Expected to Optional) > if that is not very difficult. Good point, I missed it. Should be alright to. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68723/new/ https://reviews.llvm.org/D68723 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits