[PATCH] D154328: [AST] Add API to iterate already loaded specializations

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 31 11:26:04 PDT 2023


aaron.ballman added a comment.

In D154328#4542457 <https://reviews.llvm.org/D154328#4542457>, @Hahnfeld wrote:

> Let me see if I can come up a fix for that in `clang-repl` that would then need this API. But for now, it's also fine for us if the patch stays pending in review, still better than completely disconnected local commits...

Thank you for the explanation! I think it makes sense to hold off on landing this until clang-reply needs to use the API. The code changes themselves look correct to me, so the review should hopefully sail through once there's test coverage exercising the functionality.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154328



More information about the cfe-commits mailing list