Fixed preprocessor bug which caused incorrect handling of the -D option

src/nbl/asset/utils/CWaveStringResolver.cpp

@@ -58,15 +58,20 @@ namespace nbl::wave
         // now define them as "NBL_GLSL_LIMIT_MAX_IMAGE_DIMENSION_1D=32768" 
         // to match boost wave syntax
         // https://www.boost.org/doc/libs/1_82_0/libs/wave/doc/class_reference_context.html#:~:text=Maintain%20defined%20macros-,add_macro_definition,-bool%20add_macro_definition
+
         for (const auto& define : preprocessOptions.extraDefines)
-            context.add_macro_definition(define.identifier.data() + core::string("=") + define.definition.data());
+        {
+            const std::string macroDefinition = define.identifier.data() + core::string("=") + define.definition.data();
+            const bool isMacroAdded = context.add_macro_definition(macroDefinition);
+            assert(isMacroAdded);
+        }
 
         // preprocess
         core::string resolvedString;
         try
         {
             auto stream = std::stringstream();
-            for (auto i= context.begin(); i!= context.end(); i++)
+            for (auto i = context.begin(); i != context.end(); i++)
                 stream << i->get_value();
             resolvedString = stream.str();
         }

tools/nsc/main.cpp

@@ -500,6 +500,49 @@ class ShaderCompiler final : public IApplicationFramework
         opt.debugInfoFlags = bitflag<IShaderCompiler::E_DEBUG_INFO_FLAGS>(IShaderCompiler::E_DEBUG_INFO_FLAGS::EDIF_TOOL_BIT);
         opt.dxcOptions = std::span<std::string>(m_arguments);
 
+        // need this struct becuase fields of IShaderCompiler::SMacroDefinition are string views
+        struct SMacroDefinitionBuffer
+        {
+            std::string identifier;
+            std::string definition;
+        };
+
+        core::vector<SMacroDefinitionBuffer> macroDefinitionBuffers;
+        core::vector<IShaderCompiler::SMacroDefinition> macroDefinitions;
+        {
+            for (const auto& argument : m_arguments)
+            {
+                if (argument.rfind("-D", 0) != 0)
+                    continue;
+
+                std::string argumentTmp = argument.substr(2);
+
+                std::string identifier;
+                std::string definition;
+
+                const size_t equalPos = argumentTmp.find('=');
+                if (equalPos == std::string::npos)
+                {
+                    identifier = argumentTmp;
+                    definition = "";
+                }
+                else
+                {
+                    identifier = argumentTmp.substr(0, equalPos);
+                    definition = argumentTmp.substr(equalPos + 1);
+                }
+
+                macroDefinitionBuffers.emplace_back(identifier, definition);
+            }
+
+            macroDefinitions.reserve(macroDefinitionBuffers.size());
+
+            for (const auto& macroDefinitionBuffer : macroDefinitionBuffers)
+                macroDefinitions.emplace_back(macroDefinitionBuffer.identifier, macroDefinitionBuffer.definition);
+        }
+
+        opt.preprocessorOptions.extraDefines = macroDefinitions;
+
         r.compiled = hlslcompiler->compileToSPIRV((const char*)shader->getContent()->getPointer(), opt);
         r.ok = bool(r.compiled);
         if (r.ok)