diff options
| author | Jan Svoboda <jan_svoboda@apple.com> | 2025-03-25 12:14:06 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-25 12:14:06 -0700 |
| commit | 7a370748c0928b9ccfe26127e54eb3c1a1827d75 (patch) | |
| tree | f978f04b7e0ae5f55008d635044a33d789cb711d /clang/unittests/Basic/SourceManagerTest.cpp | |
| parent | 613a077b05b8352a48695be295037306f5fca151 (diff) | |
| download | llvm-7a370748c0928b9ccfe26127e54eb3c1a1827d75.zip llvm-7a370748c0928b9ccfe26127e54eb3c1a1827d75.tar.gz llvm-7a370748c0928b9ccfe26127e54eb3c1a1827d75.tar.bz2 | |
[clang][lex] Store non-owning options ref in `HeaderSearch` (#132780)
This makes it so that `CompilerInvocation` can be the only entity that
manages ownership of `HeaderSearchOptions`, making it possible to
implement copy-on-write semantics.
Diffstat (limited to 'clang/unittests/Basic/SourceManagerTest.cpp')
| -rw-r--r-- | clang/unittests/Basic/SourceManagerTest.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/clang/unittests/Basic/SourceManagerTest.cpp b/clang/unittests/Basic/SourceManagerTest.cpp index 2b3fce9..1f2dba6 100644 --- a/clang/unittests/Basic/SourceManagerTest.cpp +++ b/clang/unittests/Basic/SourceManagerTest.cpp @@ -135,9 +135,9 @@ TEST_F(SourceManagerTest, isBeforeInTranslationUnit) { FileID mainFileID = SourceMgr.createFileID(std::move(Buf)); SourceMgr.setMainFileID(mainFileID); + HeaderSearchOptions HSOpts; TrivialModuleLoader ModLoader; - HeaderSearch HeaderInfo(std::make_shared<HeaderSearchOptions>(), SourceMgr, - Diags, LangOpts, &*Target); + HeaderSearch HeaderInfo(HSOpts, SourceMgr, Diags, LangOpts, &*Target); Preprocessor PP(std::make_shared<PreprocessorOptions>(), Diags, LangOpts, SourceMgr, HeaderInfo, ModLoader, /*IILookup =*/nullptr, @@ -185,9 +185,9 @@ TEST_F(SourceManagerTest, isBeforeInTranslationUnitWithTokenSplit) { SourceMgr.setMainFileID( SourceMgr.createFileID(llvm::MemoryBuffer::getMemBuffer(main))); + HeaderSearchOptions HSOpts; TrivialModuleLoader ModLoader; - HeaderSearch HeaderInfo(std::make_shared<HeaderSearchOptions>(), SourceMgr, - Diags, LangOpts, &*Target); + HeaderSearch HeaderInfo(HSOpts, SourceMgr, Diags, LangOpts, &*Target); Preprocessor PP(std::make_shared<PreprocessorOptions>(), Diags, LangOpts, SourceMgr, HeaderInfo, ModLoader, /*IILookup =*/nullptr, @@ -461,8 +461,8 @@ TEST_F(SourceManagerTest, loadedSLocEntryIsInTheSameTranslationUnit) { TEST_F(SourceManagerTest, ResetsIncludeLocMap) { auto ParseFile = [&] { TrivialModuleLoader ModLoader; - HeaderSearch HeaderInfo(std::make_shared<HeaderSearchOptions>(), SourceMgr, - Diags, LangOpts, &*Target); + HeaderSearchOptions HSOpts; + HeaderSearch HeaderInfo(HSOpts, SourceMgr, Diags, LangOpts, &*Target); Preprocessor PP(std::make_shared<PreprocessorOptions>(), Diags, LangOpts, SourceMgr, HeaderInfo, ModLoader, /*IILookup =*/nullptr, @@ -537,9 +537,9 @@ TEST_F(SourceManagerTest, getMacroArgExpandedLocation) { "/test-header.h", HeaderBuf->getBufferSize(), 0); SourceMgr.overrideFileContents(headerFile, std::move(HeaderBuf)); + HeaderSearchOptions HSOpts; TrivialModuleLoader ModLoader; - HeaderSearch HeaderInfo(std::make_shared<HeaderSearchOptions>(), SourceMgr, - Diags, LangOpts, &*Target); + HeaderSearch HeaderInfo(HSOpts, SourceMgr, Diags, LangOpts, &*Target); Preprocessor PP(std::make_shared<PreprocessorOptions>(), Diags, LangOpts, SourceMgr, HeaderInfo, ModLoader, @@ -656,9 +656,9 @@ TEST_F(SourceManagerTest, isBeforeInTranslationUnitWithMacroInInclude) { "/test-header.h", HeaderBuf->getBufferSize(), 0); SourceMgr.overrideFileContents(headerFile, std::move(HeaderBuf)); + HeaderSearchOptions HSOpts; TrivialModuleLoader ModLoader; - HeaderSearch HeaderInfo(std::make_shared<HeaderSearchOptions>(), SourceMgr, - Diags, LangOpts, &*Target); + HeaderSearch HeaderInfo(HSOpts, SourceMgr, Diags, LangOpts, &*Target); Preprocessor PP(std::make_shared<PreprocessorOptions>(), Diags, LangOpts, SourceMgr, HeaderInfo, ModLoader, /*IILookup =*/nullptr, |