[PATCH] D61333: [ASTImporter] Fix LLDB lookup in transparent ctx and with ext src

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 15 06:15:21 PDT 2019


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





More information about the cfe-commits mailing list