[PATCH] D48314: [Frontend] Cache preamble-related data

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 4 03:21:54 PDT 2018


ilya-biryukov added a comment.

I would argue this should be handled by the clients instead. Adding global state and locking is complicated. (And ASTUnit is complicated enough).

What are the use-cases of creating multiple `ASTUnit` inside the same process for the same file? Which clients do that and why they can't have a single ASTUnit per file?


https://reviews.llvm.org/D48314





More information about the cfe-commits mailing list