Muzi
Muzi
This PR should be merged to optimize timinig.
New modifications look good to me.
This PR might have some performance drawbacks. Since frontend timing currently satisfies the requirements, this PR is not necessary for now.
Merging #3061 and this PR into the master together will allow the CI format check to pass, which is kind of a relief as it eliminates the red cross.
> I don't think it's a good idea to limit includePaths in .scalafmt.conf, it's a project level configuration file and the IDE uses it too. > > If you want...
> > > I don't think it's a good idea to limit includePaths in .scalafmt.conf, it's a project level configuration file and the IDE uses it too. > > >...
> Q: Is there any Jia Tan behaviour here? I've only used make reformat, so I guess we all rely on scalafmt is safe.🙏
This PR should be merged to optimize timinig.
Am I correct in understanding that I may configure the top-level package as either `XiangShan` or `rocket-chip` within `cde`? This sounds OK but is rather inelegant as a submodule should...
If all the modules can be defined in the root directory, then it is the root directory that worries about its submodules instead of otherwise. This seems better because a...