diff options
| author | smanna12 <soumi.manna@intel.com> | 2023-12-22 20:39:22 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-22 20:39:22 -0600 |
| commit | bbe1b06fbb7127d613cb4958e06c737967878388 (patch) | |
| tree | 94bef85a11c2ab3caa06479573cf89bfdcd1f840 /clang/lib/CodeGen/BackendUtil.cpp | |
| parent | 61b5cc6654addf2f1c84c8c3e49f98ad6f022cb9 (diff) | |
| download | llvm-bbe1b06fbb7127d613cb4958e06c737967878388.zip llvm-bbe1b06fbb7127d613cb4958e06c737967878388.tar.gz llvm-bbe1b06fbb7127d613cb4958e06c737967878388.tar.bz2 | |
[NFC][CLANG] Fix static analyzer bugs about unnecessary object copies with auto keyword (#75082)
Reported by Static Analyzer Tool:
In ​EmitAssemblyHelper::​RunOptimizationPipeline(): Using the auto
keyword without an & causes the copy of an object of type function.
/// List of pass builder callbacks ("CodeGenOptions.h").
std::vector<std::function<void(llvm::PassBuilder &)>>
PassBuilderCallbacks;
Diffstat (limited to 'clang/lib/CodeGen/BackendUtil.cpp')
| -rw-r--r-- | clang/lib/CodeGen/BackendUtil.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/CodeGen/BackendUtil.cpp b/clang/lib/CodeGen/BackendUtil.cpp index 480410d..a6142d9 100644 --- a/clang/lib/CodeGen/BackendUtil.cpp +++ b/clang/lib/CodeGen/BackendUtil.cpp @@ -881,7 +881,7 @@ void EmitAssemblyHelper::RunOptimizationPipeline( << PluginFN << toString(PassPlugin.takeError()); } } - for (auto PassCallback : CodeGenOpts.PassBuilderCallbacks) + for (const auto &PassCallback : CodeGenOpts.PassBuilderCallbacks) PassCallback(PB); #define HANDLE_EXTENSION(Ext) \ get##Ext##PluginInfo().RegisterPassBuilderCallbacks(PB); |