================
@@ -794,6 +796,13 @@ void BackendConsumer::DontCallDiagHandler(const
DiagnosticInfoDontCall &D) {
? diag::err_fe_backend_error_attr
: diag::warn_fe_backend_warning_attr)
<< llvm::demangle(D.getFunctionName()) << D.getNote();
+
+ SmallVector<std::string> InliningDecisions = D.getInliningDecisions();
+ InliningDecisions.push_back(D.getCaller().str());
+ for (auto [index, value] : llvm::enumerate(InliningDecisions))
----------------
erichkeane wrote:
```suggestion
for (const auto &[index, value] : llvm::enumerate(InliningDecisions))
```
Seems we don't modify either, and `value` is a string, so requiring a copy of
the pair is expensive.
https://github.com/llvm/llvm-project/pull/73552
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
