martong accepted this revision.
martong added a comment.

In D59485#1446950 <https://reviews.llvm.org/D59485#1446950>, @a_sidorin wrote:

> Hello Raphael,
>  I think we should accept this change. I don't see an easy way to merge the 
> LLDB stuff into ASTImporter; also, this patch provides a good extension point 
> for ASTImporter since it is designed to be a parent class. @martong @shafik 
> Gabor, Shafik, what do you think?


Yes, I agree.


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

https://reviews.llvm.org/D59485



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to