Skip to content

Commit 38c8b00

Browse files
bognerlamb-j
authored andcommitted
Fix unused private field warning (llvm#91500)
After 11a6799 "[clang][CodeGen] Omit pre-opt link when post-opt is link requested (llvm#85672)" I'm seeing a new warning: > BackendConsumer.h:37:22: error: private field 'FileMgr' is not used [-Werror,-Wunused-private-field] Remove the field since it's no longer used. Change-Id: Ieb8531ebf020e1a46d257a4c95b0153842af542a
1 parent aaad45a commit 38c8b00

File tree

2 files changed

+11
-15
lines changed

2 files changed

+11
-15
lines changed

clang/lib/CodeGen/BackendConsumer.h

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ class BackendConsumer : public ASTConsumer {
3434
const CodeGenOptions &CodeGenOpts;
3535
const TargetOptions &TargetOpts;
3636
const LangOptions &LangOpts;
37-
const FileManager &FileMgr;
3837
std::unique_ptr<raw_pwrite_stream> AsmOutStream;
3938
ASTContext *Context;
4039
IntrusiveRefCntPtr<llvm::vfs::FileSystem> FS;
@@ -76,7 +75,7 @@ class BackendConsumer : public ASTConsumer {
7675
const PreprocessorOptions &PPOpts,
7776
const CodeGenOptions &CodeGenOpts,
7877
const TargetOptions &TargetOpts, const LangOptions &LangOpts,
79-
const FileManager &FileMgr, const std::string &InFile,
78+
const std::string &InFile,
8079
SmallVector<LinkModule, 4> LinkModules,
8180
std::unique_ptr<raw_pwrite_stream> OS, llvm::LLVMContext &C,
8281
CoverageSourceInfo *CoverageInfo = nullptr);
@@ -90,8 +89,8 @@ class BackendConsumer : public ASTConsumer {
9089
const PreprocessorOptions &PPOpts,
9190
const CodeGenOptions &CodeGenOpts,
9291
const TargetOptions &TargetOpts, const LangOptions &LangOpts,
93-
const FileManager &FileMgr, llvm::Module *Module,
94-
SmallVector<LinkModule, 4> LinkModules, llvm::LLVMContext &C,
92+
llvm::Module *Module, SmallVector<LinkModule, 4> LinkModules,
93+
llvm::LLVMContext &C,
9594
CoverageSourceInfo *CoverageInfo = nullptr);
9695

9796
llvm::Module *getModule() const;

clang/lib/CodeGen/CodeGenAction.cpp

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -112,13 +112,12 @@ BackendConsumer::BackendConsumer(
112112
const HeaderSearchOptions &HeaderSearchOpts,
113113
const PreprocessorOptions &PPOpts, const CodeGenOptions &CodeGenOpts,
114114
const TargetOptions &TargetOpts, const LangOptions &LangOpts,
115-
const FileManager &FileMgr, const std::string &InFile,
116-
SmallVector<LinkModule, 4> LinkModules,
115+
const std::string &InFile, SmallVector<LinkModule, 4> LinkModules,
117116
std::unique_ptr<raw_pwrite_stream> OS, LLVMContext &C,
118117
CoverageSourceInfo *CoverageInfo)
119118
: Diags(Diags), Action(Action), HeaderSearchOpts(HeaderSearchOpts),
120119
CodeGenOpts(CodeGenOpts), TargetOpts(TargetOpts), LangOpts(LangOpts),
121-
FileMgr(FileMgr), AsmOutStream(std::move(OS)), Context(nullptr), FS(VFS),
120+
AsmOutStream(std::move(OS)), Context(nullptr), FS(VFS),
122121
LLVMIRGeneration("irgen", "LLVM IR Generation Time"),
123122
LLVMIRGenerationRefCount(0),
124123
Gen(CreateLLVMCodeGen(Diags, InFile, std::move(VFS), HeaderSearchOpts,
@@ -138,12 +137,11 @@ BackendConsumer::BackendConsumer(
138137
const HeaderSearchOptions &HeaderSearchOpts,
139138
const PreprocessorOptions &PPOpts, const CodeGenOptions &CodeGenOpts,
140139
const TargetOptions &TargetOpts, const LangOptions &LangOpts,
141-
const FileManager &FileMgr, llvm::Module *Module,
142-
SmallVector<LinkModule, 4> LinkModules, LLVMContext &C,
143-
CoverageSourceInfo *CoverageInfo)
140+
llvm::Module *Module, SmallVector<LinkModule, 4> LinkModules,
141+
LLVMContext &C, CoverageSourceInfo *CoverageInfo)
144142
: Diags(Diags), Action(Action), HeaderSearchOpts(HeaderSearchOpts),
145143
CodeGenOpts(CodeGenOpts), TargetOpts(TargetOpts), LangOpts(LangOpts),
146-
FileMgr(FileMgr), Context(nullptr), FS(VFS),
144+
Context(nullptr), FS(VFS),
147145
LLVMIRGeneration("irgen", "LLVM IR Generation Time"),
148146
LLVMIRGenerationRefCount(0),
149147
Gen(CreateLLVMCodeGen(Diags, "", std::move(VFS), HeaderSearchOpts, PPOpts,
@@ -1082,9 +1080,8 @@ CodeGenAction::CreateASTConsumer(CompilerInstance &CI, StringRef InFile) {
10821080
std::unique_ptr<BackendConsumer> Result(new BackendConsumer(
10831081
BA, CI.getDiagnostics(), &CI.getVirtualFileSystem(),
10841082
CI.getHeaderSearchOpts(), CI.getPreprocessorOpts(), CI.getCodeGenOpts(),
1085-
CI.getTargetOpts(), CI.getLangOpts(), CI.getFileManager(),
1086-
std::string(InFile), std::move(LinkModules), std::move(OS), *VMContext,
1087-
CoverageInfo));
1083+
CI.getTargetOpts(), CI.getLangOpts(), std::string(InFile),
1084+
std::move(LinkModules), std::move(OS), *VMContext, CoverageInfo));
10881085
BEConsumer = Result.get();
10891086

10901087
// Enable generating macro debug info only when debug info is not disabled and
@@ -1245,7 +1242,7 @@ void CodeGenAction::ExecuteAction() {
12451242
BackendConsumer Result(BA, CI.getDiagnostics(), &CI.getVirtualFileSystem(),
12461243
CI.getHeaderSearchOpts(), CI.getPreprocessorOpts(),
12471244
CI.getCodeGenOpts(), CI.getTargetOpts(),
1248-
CI.getLangOpts(), CI.getFileManager(), TheModule.get(),
1245+
CI.getLangOpts(), TheModule.get(),
12491246
std::move(LinkModules), *VMContext, nullptr);
12501247

12511248
// Link in each pending link module.

0 commit comments

Comments
 (0)