[clang] [clang/DependencyScanning/ModuleDepCollector] Refactor part of `makeCommonInvocationForModuleBuild` into its own function, NFC (PR #88447)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 12 09:49:41 PDT 2024


https://github.com/jansvoboda11 approved this pull request.

LGTM. I assume you'll call `resetBenignCodeGenOptions()` from `ModuleDepCollector::applyDiscoveredDependencies()` in a follow-up, non-NFC patch, right? (So that it applies to the primary invocation, i.e. the actual PCH.) Maybe leaving a FIXME behind would be nice, just to make the intent clear.

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


More information about the cfe-commits mailing list