OpenXiangShan / XiangShan

Open-source high-performance RISC-V processor
Other
4.74k stars 646 forks source link

fix(Parameters): remove require of PaddrBits to speed up compile #3548

Closed good-circle closed 2 weeks ago

good-circle commented 2 weeks ago

In fact, our compilation speed has dropped by 75% since Sv48 was merged into the master. The reason is that this require (require(PAddrBits == 48 || !EnableSv48)) takes over 70% of the entire compile time.

This commit fixes it.

XiangShanRobot commented 2 weeks ago
[Generated by IPC robot] commit: 4df152c4c276755a7f866d931550c3ca1394726b commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
4df152c 1.865 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 2.418 3.212
master branch: commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
c8c4067 2.695 1.186 2.490 0.919 1.379 1.454 3.362 3.212
976c97c 1.855 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 2.418 3.212
9143e23 0.460 2.695 1.186 2.401 0.919 1.379 2.751 3.212
b30cb8b 1.865 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 2.418 3.212
a53daa0 1.855 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 2.418 3.212
8b2f7ab 1.865 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 2.418 3.212
dd286b6 0.460 2.695 1.186 2.822 2.490 2.401 0.919 1.379 1.454 3.362 2.751 3.212