This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG6c6abb1ded8d: [RISCV] Remove Zvamo C intrinsics and
builtins. (authored by craig.topper).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST
craig.topper updated this revision to Diff 380587.
craig.topper added a comment.
Reduce scope since RISCVISAInfo landed. We need more changes to MC tests
if we we remove Zvamo from RISCVISAInfo.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111692/n
HsiangKai accepted this revision.
HsiangKai added a comment.
This revision is now accepted and ready to land.
Zvamo is not a part of standard V extension in the current stage. I am fine to
remove it in the implementation.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https:
jrtc27 added a comment.
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
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST AC
craig.topper added a comment.
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.
arcbbb added a reviewer: jrtc27.
arcbbb added a comment.
@jrtc27 commented to keep it in https://reviews.llvm.org/D105396?id=356342
LGTM otherwise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111692/new/
https://reviews.llvm.org/D111692
___
craig.topper created this revision.
craig.topper added reviewers: kito-cheng, HsiangKai, khchen, rogfer01,
frasercrmck, arcbbb.
Herald added subscribers: achieveartificialintelligence, StephenFan, vkmr,
evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl,
jocewei, PkmX, t
