SPMI: Unify error handling path for replay and asmdiffs (#87464)
authorJakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Wed, 14 Jun 2023 07:07:18 +0000 (09:07 +0200)
committerGitHub <noreply@github.com>
Wed, 14 Jun 2023 07:07:18 +0000 (09:07 +0200)
commitd4cc3df6c9ba2212e7a1d335782abd027d6c0dd6
tree14d87e3a1bddeeb5b0785303f2aebbbc5f4d664d
parent48e1fb81858cee39be6b53b24566957aa546e891
SPMI: Unify error handling path for replay and asmdiffs (#87464)

For asmdiffs we would go into the success path always when the base JIT
succeeded. This meant that if the diff JIT missed we would confusingly
print an error message

    ERROR: method xyz is missing a compileResult, cannot do diffing

even though the "base success, diff miss" case is quite common and
expected. This change makes us take the same error path if either of the
JIT results failed, and then moves the JIT1 unique error handling into
an earlier check.
src/coreclr/tools/superpmi/superpmi/superpmi.cpp