From df655013a997cc07dde62316bd4ba0b17fb3c25a Mon Sep 17 00:00:00 2001 From: Diego Novillo Date: Wed, 16 Apr 2014 16:53:41 +0000 Subject: Allow diagnostic handlers to check for optimization remarks. Summary: When optimization remarks are enabled via the driver flag -Rpass, we should allow the FE diagnostic handler to check if the given pass name needs a diagnostic. We were unconditionally checking the pattern defined in opt's -pass-remarks flag. This was causing the FE to not emit any diagnostics. Reviewers: qcolombet CC: llvm-commits Differential Revision: http://reviews.llvm.org/D3362 llvm-svn: 206400 --- llvm/lib/IR/LLVMContext.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'llvm/lib/IR/LLVMContext.cpp') diff --git a/llvm/lib/IR/LLVMContext.cpp b/llvm/lib/IR/LLVMContext.cpp index bd87ef3ab69..588e1217bd4 100644 --- a/llvm/lib/IR/LLVMContext.cpp +++ b/llvm/lib/IR/LLVMContext.cpp @@ -130,6 +130,16 @@ void LLVMContext::diagnose(const DiagnosticInfo &DI) { pImpl->DiagnosticHandler(DI, pImpl->DiagnosticContext); return; } + + // Optimization remarks are selective. They need to check whether + // the regexp pattern, passed via -pass-remarks, matches the name + // of the pass that is emitting the diagnostic. If there is no match, + // ignore the diagnostic and return. + if (DI.getKind() == llvm::DK_OptimizationRemark && + !pImpl->optimizationRemarksEnabledFor( + cast(DI).getPassName())) + return; + // Otherwise, print the message with a prefix based on the severity. std::string MsgStorage; raw_string_ostream Stream(MsgStorage); @@ -160,8 +170,7 @@ void LLVMContext::emitOptimizationRemark(const char *PassName, const Function &Fn, const DebugLoc &DLoc, const Twine &Msg) { - if (pImpl->optimizationRemarksEnabledFor(PassName)) - diagnose(DiagnosticInfoOptimizationRemark(PassName, Fn, DLoc, Msg)); + diagnose(DiagnosticInfoOptimizationRemark(PassName, Fn, DLoc, Msg)); } //===----------------------------------------------------------------------===// -- cgit v1.2.3