[clang] [clang] Do not share ownership of `HeaderSearchOptions` (PR #132984)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 28 11:32:52 PDT 2025


================
@@ -116,7 +116,7 @@ class ASTUnit {
   std::shared_ptr<Preprocessor>           PP;
   IntrusiveRefCntPtr<ASTContext>          Ctx;
   std::shared_ptr<TargetOptions>          TargetOpts;
-  std::shared_ptr<HeaderSearchOptions>    HSOpts;
+  std::unique_ptr<HeaderSearchOptions> HSOpts;
----------------
jansvoboda11 wrote:

Not really, I just didn't want to change the size of `ASTUnit` drastically (`HeaderSearchOptions` is 576B).

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


More information about the cfe-commits mailing list