sammccall added a comment. In https://reviews.llvm.org/D50958#1212179, @ilya-biryukov wrote:
> Parts of this patch were landed as https://reviews.llvm.org/D50847 and > https://reviews.llvm.org/D50889, happy to discuss the design decisions > ("merged" dynamic index, two callbacks), but that's probably out of scope of > this patch. Ah ok, that explains some of the unrelated changes. I do have some questions there that would be good to discuss. Meanwhile, @hokein can you rebase this patch against head? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D50958 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
