Dev btb rebase #308
Open
Dev btb rebase #308
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add support for DecoupledBPUWithBTB in xiangshan.py configuration - Disable loop predictor, loop buffer, and jump ahead predictor for DecoupledBPUWithBTB - Add enableSC parameter to BTBTAGE branch predictor
jensen-yan
force-pushed
the
dev-btb-rebase
branch
from
7ee27fe to
d021465
Compare
- Add comprehensive README.cn.md for branch predictor module - Provide detailed documentation for DecoupledBPUWithBTB architecture - Include key data structures, prediction flow, and special features - Add inline documentation for critical header files - Create script for Kunming Lake 6-wide BTB configuration
- Remove hardcoded getDelay() methods in BTBTAGE, BTBITTAGE, DefaultBTB, and BTBRAS - Add numDelay parameter to TimedBaseBTBPredictor base class - Update BranchPredictor.py to use parent block size and set default numDelay values - Simplify predictor configuration by using a common delay mechanism
tastynoob
approved these changes
Mar 5, 2025
wiggleman
approved these changes
Mar 5, 2025
not use fixed 32 byte or 0x20 byte, now we can change it in xiangshan.py, use 64byte, blockSize is equal to predictWidth using in FTB.
- Introduce `alignToBlockSize` parameter to control BTB prediction alignment - Modify `stream_struct.hh` to handle fall-through address calculation based on alignment - Update `btb.cc` to set index shift amount dynamically based on alignment - Add configuration options in `xiangshan.py` for DecoupledBPUWithBTB - Create script `kmh_v3_btb.sh` for BTB-based branch prediction configuration
|
Very difficult to review. May delay a few days if you want me to review the code. |
- Usage: scons build/RISCV/compile_commands.json - Ouput: in build/RISCV/compile_commands.json - Enable SCons compilation database generation for all build variants - Generate `compile_commands.json` in each variant directory - Use absolute paths for better IDE integration Change-Id: I1efb12ffd09c889235c5bea2860ca3d373701a50
- Implement comprehensive test suite for μRAS predictor using Google Test framework - Add MockuRAS class to simulate core RAS functionality - Create test cases covering: * Basic push and pop operations * Counter behavior * Stack full scenarios * Speculative update history * Prediction recovery mechanisms - Add SConscript for test compilation - Include detailed README.md explaining test suite structure and usage This commit enhances the branch predictor testing infrastructure by providing a robust test framework for the μRAS predictor.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
rebase dev-btb to the master
use --bp-type=DecoupledBPUWithBTB
to use btb BPU