diff options
author | Jan Svoboda <jan_svoboda@apple.com> | 2025-10-02 11:01:27 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-10-02 11:01:27 -0700 |
commit | b86ddae1da651f921125a9864b45a5b11bc3b1c0 (patch) | |
tree | 4ec04e58eb5d9b78109c171fc83ede76843027e8 /llvm/lib/CodeGen/StackFrameLayoutAnalysisPass.cpp | |
parent | 99d85906c542c3801a24137ba6d6f2c367308563 (diff) | |
download | llvm-b86ddae1da651f921125a9864b45a5b11bc3b1c0.zip llvm-b86ddae1da651f921125a9864b45a5b11bc3b1c0.tar.gz llvm-b86ddae1da651f921125a9864b45a5b11bc3b1c0.tar.bz2 |
[clang] NFCI: Clean up `CompilerInstance::create{File,Source}Manager()` (#160748)
The `CompilerInstance::createSourceManager()` function currently accepts
the `FileManager` to be used. However, all clients call
`CompilerInstance::createFileManager()` prior to creating the
`SourceManager`, and it never makes sense to use a `FileManager` in the
`SourceManager` that's different from the rest of the compiler. Passing
the `FileManager` explicitly is redundant, error-prone, and deviates
from the style of other `CompilerInstance` initialization APIs.
This PR therefore removes the `FileManager` parameter from
`createSourceManager()` and also stops returning the `FileManager`
pointer from `createFileManager()`, since that was its primary use. Now,
`createSourceManager()` internally calls `getFileManager()` instead.
Diffstat (limited to 'llvm/lib/CodeGen/StackFrameLayoutAnalysisPass.cpp')
0 files changed, 0 insertions, 0 deletions