Mesa (master): radeonsi: don't print diagnostic LLVM remarks and notes

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Tue Oct 29 01:37:23 UTC 2019


Module: Mesa
Branch: master
Commit: 1380db9fa8dc5a519a35394ec4fa6e5953e00ffe
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1380db9fa8dc5a519a35394ec4fa6e5953e00ffe

Author: Marek Olšák <marek.olsak at amd.com>
Date:   Thu Oct 24 00:22:58 2019 -0400

radeonsi: don't print diagnostic LLVM remarks and notes

We don't use them.

Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

---

 src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c b/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c
index 1c6522322ff..91e9bd3dd68 100644
--- a/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c
+++ b/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c
@@ -44,7 +44,6 @@ static void si_diagnostic_handler(LLVMDiagnosticInfoRef di, void *context)
 {
 	struct si_llvm_diagnostics *diag = (struct si_llvm_diagnostics *)context;
 	LLVMDiagnosticSeverity severity = LLVMGetDiagInfoSeverity(di);
-	char *description = LLVMGetDiagInfoDescription(di);
 	const char *severity_str = NULL;
 
 	switch (severity) {
@@ -55,15 +54,13 @@ static void si_diagnostic_handler(LLVMDiagnosticInfoRef di, void *context)
 		severity_str = "warning";
 		break;
 	case LLVMDSRemark:
-		severity_str = "remark";
-		break;
 	case LLVMDSNote:
-		severity_str = "note";
-		break;
 	default:
-		severity_str = "unknown";
+		return;
 	}
 
+	char *description = LLVMGetDiagInfoDescription(di);
+
 	pipe_debug_message(diag->debug, SHADER_INFO,
 			   "LLVM diagnostic (%s): %s", severity_str, description);
 




More information about the mesa-commit mailing list