Updated glslang, SPIRV-Cross, and SPIRV-Tools.

Replaced usage of the removed SPIRV remapper from glslang with equivalents
in SPIRV-Tool's optimizer. Also replaced spirv.hpp to that brought in
from SPIRV-Headers.

Use UTF-8 codepage for Windows, allowing for using Unicode paths similar to
what is currently done on Mac and Linux.

Avoid printing full output on invalid arguments. This makes it easier to
see what the error is for a simple mistake. Running tools by themselves
will still show the full help output, but now without other errors, though
will still have a non-zero exit code to indicate that it wasn't a
successful run.

Author: akb825

CMakeLists.txt

@@ -90,7 +90,7 @@ endif()
 
 set(MSL_MAJOR_VERSION 1)
 set(MSL_MINOR_VERSION 8)
-set(MSL_PATCH_VERSION 1)
+set(MSL_PATCH_VERSION 2)
 set(MSL_VERSION ${MSL_MAJOR_VERSION}.${MSL_MINOR_VERSION}.${MSL_PATCH_VERSION})
 
 set(MSL_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR})

Compile/SPIRV-Cross

@@ -34,7 +34,6 @@
 #endif
 
 #include "SPIRV/GlslangToSpv.h"
-#include "SPIRV/SPVRemapper.h"
 #include "glslang/Public/ResourceLimits.h"
 
 #if MSL_GCC || MSL_CLANG
@@ -344,30 +343,26 @@ void Compiler::process(SpirV& spirv, int processOptions)
 	if (processOptions == 0)
 		return;
 
-	std::uint32_t options = 0;
+	// NOTE: We have some known invalid code, such as missing bindings. Therefore we need to skip
+	// validation.
+	spv_optimizer_options options = spvOptimizerOptionsCreate();
+	spvOptimizerOptionsSetRunValidator(options, false);
+	spvtools::Optimizer optimizer(SPV_ENV_VULKAN_1_0);
 	if (processOptions & RemapVariables)
-		options |= spv::spirvbin_t::MAP_ALL;
+		optimizer.RegisterPass(spvtools::CreateCanonicalizeIdsPass());
 	if (processOptions & DeadCodeElimination)
-		options |= spv::spirvbin_t::DCE_ALL;
+	{
+		optimizer.RegisterPass(spvtools::CreateEliminateDeadFunctionsPass());
+		optimizer.RegisterPass(spvtools::CreateEliminateDeadConstantPass());
+	}
 	if (processOptions & StripDebug)
-		options |= spv::spirvbin_t::STRIP;
-
-	spv::spirvbin_t remapper;
-	remapper.remap(spirv, options);
-
+		optimizer.RegisterPass(spvtools::CreateStripDebugInfoPass());
 	if (processOptions & Optimize)
-	{
-		// NOTE: We have some known invalid code, such as missing bindings. Therefore we need to
-		// skip validation.
-		spv_optimizer_options options = spvOptimizerOptionsCreate();
-		spvOptimizerOptionsSetRunValidator(options, false);
-		spvtools::Optimizer optimizer(SPV_ENV_VULKAN_1_0);
 		optimizer.RegisterPerformancePasses();
-		SpirV optimizedSpirV;
-		if (optimizer.Run(spirv.data(), spirv.size(), &optimizedSpirV, options))
-			spirv = std::move(optimizedSpirV);
-		spvOptimizerOptionsDestroy(options);
-	}
+	SpirV optimizedSpirV;
+	if (optimizer.Run(spirv.data(), spirv.size(), &optimizedSpirV, options))
+		spirv = std::move(optimizedSpirV);
+	spvOptimizerOptionsDestroy(options);
 }
 
 } // namespace msl

Compile/SPIRV-Tools

@@ -15,9 +15,19 @@
  */
 
 #include "GlslOutput.h"
-#include "spirv_glsl.hpp"
 #include <MSL/Compile/Output.h>
 
+#if MSL_GCC || MSL_CLANG
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wconversion"
+#endif
+
+#include "spirv_glsl.hpp"
+
+#if MSL_GCC || MSL_CLANG
+#pragma GCC diagnostic pop
+#endif
+
 namespace msl
 {
 

Compile/glslang

@@ -15,9 +15,19 @@
  */
 
 #include "MetalOutput.h"
-#include "spirv_msl.hpp"
 #include <MSL/Compile/Output.h>
 
+#if MSL_GCC || MSL_CLANG
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wconversion"
+#endif
+
+#include "spirv_msl.hpp"
+
+#if MSL_GCC || MSL_CLANG
+#pragma GCC diagnostic pop
+#endif
+
 namespace msl
 {
 

Compile/src/Compiler.cpp

@@ -17,7 +17,7 @@
 #include "SpirVProcessor.h"
 #include <MSL/Compile/CompiledResult.h>
 #include <MSL/Compile/Output.h>
-#include <SPIRV/spirv.hpp>
+#include <spirv/unified1/spirv.hpp>
 #include <algorithm>
 #include <cassert>
 #include <cstring>

Compile/src/GlslOutput.cpp

@@ -29,7 +29,7 @@
 #include <boost/algorithm/string/classification.hpp>
 #include <boost/algorithm/string/split.hpp>
 #include <boost/algorithm/string/trim.hpp>
-#include <SPIRV/spirv.hpp>
+#include <spirv/unified1/spirv.hpp>
 
 #include <algorithm>
 #include <cassert>

Compile/src/MetalOutput.cpp

@@ -3,6 +3,11 @@ find_package(Boost CONFIG COMPONENTS program_options filesystem)
 file(GLOB_RECURSE sources *.cpp *.h)
 add_executable(mslb-extract ${sources})
 
+if (WIN32)
+	configure_file(mslb-extract.manifest.in ${CMAKE_CURRENT_BINARY_DIR}/mslb-extract.manifest @ONLY)
+	target_sources(mslb-extract PRIVATE ${CMAKE_CURRENT_BINARY_DIR}/mslb-extract.manifest)
+endif()
+
 target_link_libraries(mslb-extract PRIVATE MSL::Client Boost::program_options Boost::filesystem)
 target_compile_definitions(mslb-extract PRIVATE BOOST_ALL_NO_LIB
 	MSL_MAJOR_VERSION=${MSL_MAJOR_VERSION} MSL_MINOR_VERSION=${MSL_MINOR_VERSION}