From 306cf7fcc0b05236583b08ff15f7726e5ecfb978 Mon Sep 17 00:00:00 2001
From: baldurk <baldurk@baldurk.org>
Date: Sun, 4 Oct 2015 13:13:10 +0200
Subject: [PATCH] Tweak scalar/vector constant display to be a little nicer

---
 .../shaders/spirv/spirv_disassemble.cpp       | 29 +++++++++++++++++--
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/renderdoc/driver/shaders/spirv/spirv_disassemble.cpp b/renderdoc/driver/shaders/spirv/spirv_disassemble.cpp
index 55906a8f8..7f9b50a6f 100644
--- a/renderdoc/driver/shaders/spirv/spirv_disassemble.cpp
+++ b/renderdoc/driver/shaders/spirv/spirv_disassemble.cpp
@@ -371,11 +371,11 @@ struct SPVConstant
 		if(type->type == SPVTypeData::eFloat)
 		{
 			if(type->bitCount == 64)
-				return StringFormat::Fmt("%lf", d);
+				return StringFormat::Fmt("%lgf", d);
 			if(type->bitCount == 32)
-				return StringFormat::Fmt("%f", f);
+				return StringFormat::Fmt("%gf", f);
 			if(type->bitCount == 16)
-				return StringFormat::Fmt("%f", ConvertFromHalf(u16));
+				return StringFormat::Fmt("%gf", ConvertFromHalf(u16));
 		}
 		else if(type->type == SPVTypeData::eSInt)
 		{
@@ -412,6 +412,29 @@ struct SPVConstant
 			return GetValString();
 		}
 
+		// special case vectors with the same constant
+		// replicated across all channels
+		if(type->type == SPVTypeData::eVector)
+		{
+			bool identical = true;
+			for(size_t i=1; i < children.size(); i++)
+			{
+				if(children[i]->u64 != children[0]->u64)
+				{
+					identical = false;
+					break;
+				}
+			}
+
+			if(identical)
+			{
+				string ret = children[0]->GetValString() + ".";
+				for(size_t i=0; i < children.size(); i++)
+					ret += 'x';
+				return ret;
+			}
+		}
+
 		string ret = type->GetName();
 		ret += "(";
 		if(children.empty())