Skip to content

feat(BusKeyField): add reqSource id of L2 next line (nl) prefetcher#139

Merged
zzQGyy merged 8 commits intomasterfrom
feat_add_nl_prefetcher
Mar 10, 2026
Merged

feat(BusKeyField): add reqSource id of L2 next line (nl) prefetcher#139
zzQGyy merged 8 commits intomasterfrom
feat_add_nl_prefetcher

Conversation

@zzQGyy
Copy link
Copy Markdown
Contributor

@zzQGyy zzQGyy commented Mar 4, 2026

Added the nl id in MemReqSource, and added a check for nl req in def isL2Prefetch

Copy link
Copy Markdown
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

Adds a new MemReqSource enumeration entry to represent the L2 next-line (NL) prefetcher and ensures it is recognized by the isL2Prefetch helper used for request-source classification.

Changes:

  • Add Prefetch2L2NL to MemReqSource.
  • Treat Prefetch2L2NL as an L2 prefetch in isL2Prefetch.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/scala/utility/TLUtils/BusKeyField.scala Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@zzQGyy zzQGyy merged commit 84a74fc into master Mar 10, 2026
1 check passed
@zzQGyy zzQGyy deleted the feat_add_nl_prefetcher branch March 10, 2026 02:22
rich-cake pushed a commit to OpenXiangShan/XiangShan that referenced this pull request Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants