================ @@ -1909,7 +1909,19 @@ class Sema final : public SemaBase { /// '\#pragma clang attribute push' directives to the given declaration. void AddPragmaAttributes(Scope *S, Decl *D); - void PrintPragmaAttributeInstantiationPoint(); + using DiagFuncRef = + llvm::function_ref<void(SourceLocation, PartialDiagnostic)>; + auto getDefaultDiagFunc() { + return [this](SourceLocation Loc, PartialDiagnostic PD) { ---------------- erichkeane wrote:
Ah, sorry I didn't clarify. I'm fine not using `PartialDiagnosticAt`, I've had a similar concern with a different patch I was working on. I was mostly answering your question/exposition about the source locations. https://github.com/llvm/llvm-project/pull/125453 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits