Zvamo is not part of the 1.0 V spec. Remove it for now.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
@jrtc27 commented to keep it in https://reviews.llvm.org/D105396?id=356342
LGTM otherwise.
Comment Actions
My motivation was to reduce clang binary size by at least 50K and reduce our intrinsic count by 4% while we figure out ways to reduce the RISCV vector intrinsic load on clang.
Comment Actions
I don't have a problem with deleting the code if there's a technical justification, just wanted to avoid churn if it was purely for "it's not ratified" reasons and we think it'll reappear in another form later
Comment Actions
Zvamo is not a part of standard V extension in the current stage. I am fine to remove it in the implementation.
Comment Actions
Reduce scope since RISCVISAInfo landed. We need more changes to MC tests
if we we remove Zvamo from RISCVISAInfo.