SPMI: Fix wrong SPMI replay result check (#93062)

jakobbotsch · web-flow · commit 0b4ec5a1e460 · 2023-10-05T17:15:38.000+02:00
Fix a bug introduced in #92824 that would SPMI to not show replay failures.
diff --git a/src/coreclr/tools/superpmi/superpmi/superpmi.cpp b/src/coreclr/tools/superpmi/superpmi/superpmi.cpp
@@ -441,7 +441,7 @@ int __cdecl main(int argc, char* argv[])
                 mc->cr->dumpToConsole(); // Dump the compile results if doing debug logging
             }
         }
-        else if (res.Result == ReplayResult::Success)
+        else if (res.Result == ReplayResult::Error)
         {
             errorCount++;
             LogError("Method %d of size %d failed to load and compile correctly%s (%s).",

Original file line number	Diff line number	Diff line change
`@@ -441,7 +441,7 @@ int __cdecl main(int argc, char* argv[])`
`441`	`441`	`mc->cr->dumpToConsole(); // Dump the compile results if doing debug logging`
`442`	`442`	`}`
`443`	`443`	`}`
`444`		`- else if (res.Result == ReplayResult::Success)`
	`444`	`+ else if (res.Result == ReplayResult::Error)`
`445`	`445`	`{`
`446`	`446`	`errorCount++;`
`447`	`447`	`LogError("Method %d of size %d failed to load and compile correctly%s (%s).",`