Skip to content

Conversation

jkoritzinsky
Copy link
Member

These headers were completely unused, delete them.

@Copilot Copilot AI review requested due to automatic review settings October 17, 2025 18:22
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Removes unused dead code and associated headers to reduce maintenance surface.

  • Deletes specialstatics.h and bitmask.{h,inl} along with an unused STATIC_DATA::EnumMemoryRegions implementation.
  • Drops now-unneeded include directives referencing the removed headers.

Reviewed Changes

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

Show a summary per file
File Description
src/coreclr/vm/threads.cpp Removes unused STATIC_DATA::EnumMemoryRegions definition.
src/coreclr/vm/specialstatics.h Deletes obsolete STATIC_DATA structure header.
src/coreclr/vm/object.h Removes include of deleted specialstatics.h.
src/coreclr/vm/common.h Removes include of deleted specialstatics.h.
src/coreclr/vm/ceeload.h Removes include of deleted bitmask.h.
src/coreclr/inc/bitmask.inl Deletes unused BitMask inline implementation.
src/coreclr/inc/bitmask.h Deletes unused BitMask and SynchronizedBitMask declarations.

Copy link
Contributor

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

@jkoritzinsky jkoritzinsky enabled auto-merge (squash) October 17, 2025 20:51
@jkoritzinsky jkoritzinsky merged commit 3ef4563 into dotnet:main Oct 17, 2025
99 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants