commit 9a6e043512a32c67525715af707a9ea75139ef7e
author Chris Forbes <chrisforbes@google.com> Thu Aug 15 19:39:29 2019 -0700
committer Chris Forbes <chrisforbes@google.com> Mon Aug 19 16:44:26 2019 +0000
tree 7cfecafee53e892170744405e20acb3f68bc355e
parent d335302a7c0bb24f4427586d2fa294764cd0777a

Add newlines to SPIRV optimizer messages

vk::warn etc expects the client to include the newline
Change-Id: Ib0f808d7d397c37e8c52d7ce1977551a206dbc40
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/35251
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Tested-by: Chris Forbes <chrisforbes@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>

src/Vulkan/VkPipeline.cpp

diff --git a/src/Vulkan/VkPipeline.cpp b/src/Vulkan/VkPipeline.cpp
index cdfc887..5558223 100644
--- a/src/Vulkan/VkPipeline.cpp
+++ b/src/Vulkan/VkPipeline.cpp
@@ -165,13 +165,13 @@
 	opt.SetMessageConsumer([](spv_message_level_t level, const char*, const spv_position_t& p, const char* m) {
 		switch (level)
 		{
-		case SPV_MSG_FATAL:          vk::warn("SPIR-V FATAL: %d:%d %s", int(p.line), int(p.column), m);
-		case SPV_MSG_INTERNAL_ERROR: vk::warn("SPIR-V INTERNAL_ERROR: %d:%d %s", int(p.line), int(p.column), m);
-		case SPV_MSG_ERROR:          vk::warn("SPIR-V ERROR: %d:%d %s", int(p.line), int(p.column), m);
-		case SPV_MSG_WARNING:        vk::warn("SPIR-V WARNING: %d:%d %s", int(p.line), int(p.column), m);
-		case SPV_MSG_INFO:           vk::trace("SPIR-V INFO: %d:%d %s", int(p.line), int(p.column), m);
-		case SPV_MSG_DEBUG:          vk::trace("SPIR-V DEBUG: %d:%d %s", int(p.line), int(p.column), m);
-		default:                     vk::trace("SPIR-V MESSAGE: %d:%d %s", int(p.line), int(p.column), m);
+		case SPV_MSG_FATAL:          vk::warn("SPIR-V FATAL: %d:%d %s\n", int(p.line), int(p.column), m);
+		case SPV_MSG_INTERNAL_ERROR: vk::warn("SPIR-V INTERNAL_ERROR: %d:%d %s\n", int(p.line), int(p.column), m);
+		case SPV_MSG_ERROR:          vk::warn("SPIR-V ERROR: %d:%d %s\n", int(p.line), int(p.column), m);
+		case SPV_MSG_WARNING:        vk::warn("SPIR-V WARNING: %d:%d %s\n", int(p.line), int(p.column), m);
+		case SPV_MSG_INFO:           vk::trace("SPIR-V INFO: %d:%d %s\n", int(p.line), int(p.column), m);
+		case SPV_MSG_DEBUG:          vk::trace("SPIR-V DEBUG: %d:%d %s\n", int(p.line), int(p.column), m);
+		default:                     vk::trace("SPIR-V MESSAGE: %d:%d %s\n", int(p.line), int(p.column), m);
 		}
 	});