martong added a comment. In D61333#1583173 <https://reviews.llvm.org/D61333#1583173>, @teemperor wrote:
> @martong Sorry for the delay, feel free to ping me in the future about these > patches. I'll review them ASAP now that I'm back in office, so these delay's > hopefully won't happen again. > > I tried applying this patch and it seems it needs to be rebased. I would do > it myself, but I'm not entirely sure how to rebase the changes to > `ASTNodeImporter::ImportDefinition`. It seems we got rid of > `To->completeDefinition();`, so not sure if the code that this patch adds is > still in the right place. Raphael, Thank you for looking into this. I've rebased to master and updated the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61333/new/ https://reviews.llvm.org/D61333 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
