Skip to content

Replace PERFSCORE memory access kind macros with enum class#127046

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/replace-memory-read-kinds-with-enum
Draft

Replace PERFSCORE memory access kind macros with enum class#127046
Copilot wants to merge 3 commits intomainfrom
copilot/replace-memory-read-kinds-with-enum

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 17, 2026

Summary

  • replace PERFSCORE_MEMORY_* macro constants in src/coreclr/jit/emit.h with a strongly-typed PerfScoreMemoryAccessKind enum class
  • rename enum members to CamelCase without the legacy prefix: None, Read, Write, and ReadWrite
  • strongly type insExecutionCharacteristics.insMemoryAccessKind as PerfScoreMemoryAccessKind
  • update affected emitter call sites and comparisons to use qualified enum values (PerfScoreMemoryAccessKind::None, ::Read, ::Write, ::ReadWrite) across xarch/arm64/loongarch64/riscv64 and shared perfscore logic
  • update arm64 helper signature to use PerfScoreMemoryAccessKind* instead of unsigned*

Validation

  • ./build.sh clr+libs+host (baseline)
  • ./build.sh clr.jit -rc release
  • ./build.sh clr.jit -rc release -test
  • parallel_validation (Code Review: no comments, CodeQL: no alerts; cpp analysis skipped due database size)

Note

This pull request description was generated with AI assistance.

Copilot AI requested review from Copilot and removed request for Copilot April 17, 2026 05:19
Copilot AI requested a review from jakobbotsch April 17, 2026 05:20
@github-actions github-actions bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Apr 17, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

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

Comment thread src/coreclr/jit/emit.h Outdated
Copilot AI requested review from Copilot and removed request for Copilot April 17, 2026 07:46
Copilot AI changed the title Replace PERFSCORE memory access kind macros with enum Replace PERFSCORE memory access kind macros with enum class Apr 17, 2026
Copilot AI requested a review from jakobbotsch April 17, 2026 07:50
Comment thread src/coreclr/jit/emit.h Outdated
Copilot AI requested review from Copilot and removed request for Copilot April 17, 2026 08:36
Copilot AI requested a review from jakobbotsch April 17, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants