Delete dead code

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426

[jkotas]

No description provided.

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

src/coreclr/vm/threads.cpp:7158

• The removal of FinalizerBase and consolidation into KickOff for both thread types is clear dead code deletion. Please ensure relevant tests cover the finalizer thread behavior to verify that exception handling and thread kickoff work as expected.

void ManagedThreadBase::KickOff(ADCallBackFcnType pTarget, LPVOID args)

src/coreclr/vm/excep.cpp:4541

• The call to DefaultCatchHandler has been modified to remove the 'isThreadBaseFilter' and 'sendAppDomainEvents' parameters. Ensure that this change aligns with the new function interface and that the expected exception event handling behavior is maintained.

DefaultCatchHandler(pParam->pExceptionInfo, NULL, useLastThrownObject, TRUE /* sendWindowsEventLog */);

src/coreclr/vm/ceeload.cpp:489

• The removal of the DACCESS_COMPILE guard around the initialization of m_pJitInlinerTrackingMap is observed. Confirm that this change is intentional and that it does not affect builds targeting DACCESS.

#if !defined(DACCESS_COMPILE)

src/coreclr/vm/appdomain.hpp:1148

• Changing the signature of OnUnhandledException from returning BOOL to void is a breaking change. Verify that all call sites have been updated to accommodate the new signature and that behavior remains consistent.

static void OnUnhandledException(OBJECTREF *pThrowable);

[dotnet-policy-service]

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

[jkotas]

ManagedThread and FinalizerThread were the only two actually used values from this enum, and the one place that they were used was dead code

[jkotas]

This was always called with swallowing = true, so the rest of this method is dead code. The small piece from the above (ExceptionIsAlwaysSwallowed check) was folded into the caller.

[janvorli]

LGTM! Thank you!

[janvorli]

/ba-g the test failure has been happening consistently on all runs since yesterday. #114426