Skip to content

Remove dead code#120852

Merged
jkoritzinsky merged 2 commits intodotnet:mainfrom
jkoritzinsky:dead-code-removal
Oct 17, 2025
Merged

Remove dead code#120852
jkoritzinsky merged 2 commits intodotnet:mainfrom
jkoritzinsky:dead-code-removal

Conversation

@jkoritzinsky
Copy link
Member

These headers were completely unused, delete them.

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

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.

@dotnet-policy-service
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
@github-actions github-actions bot locked and limited conversation to collaborators Nov 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants