[llvm] [SPIRV] Emit NonSemantic DebugValue (PR #224158)

[Manuel Carrasco via llvm-commits]

================
@@ -1402,14 +1713,14 @@ void SPIRVNonSemanticDebugHandler::beginInstruction(const MachineInstr *MI) {
   if (!DebugFunctionDefinitionEmitted)
     return;
 
+  emitAnalyzedRecords(MI);
+
   std::optional<const MachineInstr *> Target = resolveDebugLocTarget(MI);
   if (!Target)
     return;
 
-  emitDebugScopeForInstruction(*Target);
+  (void)emitDebugScopeForInstruction(*Target);
   emitDebugLineForInstruction(*Target);
-
-  emitDebugDeclare(MI);
----------------
mgcarrasco wrote:

Just to understand, why this is now handled differently? Was it buggy or it was just refactoring since it had similar constraints as dbg_value? 

https://github.com/llvm/llvm-project/pull/224158