hokein added a comment.

In https://reviews.llvm.org/D44882#1065932, @malaperle wrote:

> In https://reviews.llvm.org/D44882#1065787, @hokein wrote:
>
> > @malaperle, what's your plan of this patch? Are you going to land it before 
> > https://reviews.llvm.org/D45513? With the Line&Column info in the index, 
> > this patch could be simplified.
>
>
> I'll address the last comment and wait for review. Probably at least another 
> day of delay. So if that ends up after https://reviews.llvm.org/D45513 then 
> I'll update it to simplify. Sounds good?


Thanks, I have landed the patch today, you need to update your patch (I think 
it is mostly about removing the code of reading-file stuff).


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44882



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

Reply via email to