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).

  rCTE Clang Tools Extra


cfe-commits mailing list

Reply via email to