shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land.
LGTM ================ Comment at: lldb/source/Symbol/ClangASTImporter.cpp:1006 +/// Takes a CXXMethodDecl and completes the return type if necessary. This +/// is currently only necessary for virtual functions with covariant return +/// types where Clang's CodeGen expects that the underlying records are already ---------------- I am curious if we foresee the need to do this for other cases. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73024/new/ https://reviews.llvm.org/D73024 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
