[clang] [clang] Move state out of `PreprocessorOptions` (1/n) (PR #86358)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 28 13:59:41 PDT 2024


================
@@ -34,12 +34,17 @@ class InitOnlyAction : public FrontendAction {
 
 /// Preprocessor-based frontend action that also loads PCH files.
 class ReadPCHAndPreprocessAction : public FrontendAction {
+  llvm::function_ref<void(CompilerInstance &)> OnCI;
----------------
jansvoboda11 wrote:

The function acts on `CompilerInstance`, so I figured `OnCI` fits. But I guess it's a bit too terse. How about `AdjustCI`? It's basically here to be able to tweak the `CompilerInstance` in a way that `CompilerInvocation` alone can't (and neither can this `FrontendAction` without some additional context).

https://github.com/llvm/llvm-project/pull/86358


More information about the cfe-commits mailing list