[PATCH] D136684: [clang][ASTImporter] Remove use of ParentMapContext.

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 25 08:37:25 PDT 2022


martong accepted this revision.
martong added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: rnkovacs.

It is very good that we can get rid of the `ParentMapContext` because it was sub-optimal to build that up for **//all//** declaration contexts of the translation unit. Now we discover the parent-child relations for ourselves and only for the needed declaration contexts. This is the proper way to go.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136684



More information about the cfe-commits mailing list