hokein added a comment.

In D74610#1879054 <https://reviews.llvm.org/D74610#1879054>, @kadircet wrote:

> LGTM,
>
> just wondering though if you've tried the solution we discussed offline 
> regarding updating `indexTypeSourceInfo` instead of this call site, could you 
> mention the results in here?
>  So that the next soul that touches these lines has an idea about what's 
> expecting them?


It seemed breaking a few existing tests...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74610



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

Reply via email to