[release/8.0-staging] Fix generation of minidump (#115562)

[tommcdon]

Ports #115562

Customer Impact

• Customer reported
• Found internally

Addresses a customer reported issue in minidump generation. The DAC implements ICLRDataEnumMemoryRegions::EnumMemoryRegions which is used by dbghelp!MiniDumpWriteDump to mark pages in the process that must be saved for actionable dumps. As some memory enumeration API's might fail at dump collection time, we implement try/catch blocks around the logic that collects memory pages so that the entire process is not aborted. We have a customer report on an issue resulting in corrupted mini-dumps, missing memory needed for stackwalking. The issue was narrowed down to a call to GetIL on an InlinedCallFrame causes an exception to be thrown during memory enumeration, resulting in skipping over other memory collection API's that would have succeeded and in turn creating a minidump missing memory needed for stackwalking. The fix addresses the issue by wrapping the IL version method calls at a lower-level with a nested try-catch block to ensure that an exception in GetIL doesn’t prevent dumping a full callstack.

Regression

• Yes (Works well on .NET Framework)
• No

Testing

Manually tested, the customer verified

Risk

Low risk, we are adding a more granular try/catch around dump generation code

[Copilot]

Pull Request Overview

This PR aims to address a customer-reported issue where an exception in GetIL causes incomplete minidump generation. The changes add a nested try/catch block around IL version method calls to ensure that an exception from GetIL does not prevent processing of other memory regions.

• Added a try/catch block around IL version method calls.
• Restricted exception rethrowing to COR_E_OPERATIONCANCELLED.

Comments suppressed due to low confidence (1)

src/coreclr/vm/method.cpp:3797

• [nitpick] The nested try/catch block around IL version calls improves minidump reliability, but please confirm that handling only COR_E_OPERATIONCANCELLED is sufficient for this context. It may be helpful to document the reasoning for limiting the catch to this exception type.

              ilVersion.GetIL();

[tommcdon]

Closing as a dupe to #115739