arcbbb planned changes to this revision.
arcbbb added a comment.
Herald added a subscriber: StephenFan.
I started a discussion on how RVV clang builtins interact with FENV_ACCESS in
https://github.com/riscv-non-isa/rvv-intrinsic-doc/issues/147
I would drop this patch if those builtins are going t
arcbbb created this revision.
arcbbb added reviewers: rsandifo-arm, efriedma, sdesmalen, rovka, rjmccall,
rengolin, HsiangKai, craig.topper.
Herald added subscribers: vkmr, frasercrmck, evandro, luismarques, apazos,
sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o,
brucehoult,
arcbbb updated this revision to Diff 349600.
arcbbb marked an inline comment as done.
arcbbb retitled this revision from "[Sema][RISCV] Allow ?: to select Typedef
BuiltinType in C" to "[Sema][RISCV][SVE] Allow ?: to select Typedef BuiltinType
in C".
arcbbb added a reviewer: frasercrmck.
arcbbb ad
This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rGfcf8827a98be: [Sema][RISCV][SVE] Allow ?: to select Typedef
BuiltinType in C (authored by arcbbb).
Repository:
rG LLVM
arcbbb created this revision.
arcbbb added reviewers: rogfer01, frasercrmck, jrtc27, craig.topper, evandro,
HsiangKai, khchen, kito-cheng.
Herald added subscribers: vkmr, dexonsmith, jdoerfert, luismarques, apazos,
sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, jfb, the_o,
bruceho
arcbbb updated this revision to Diff 356353.
arcbbb retitled this revision from "[RISCV] Remove Zvamo extension for v1.0-rc
change" to "[RISCV] Remove Zvamo implication for v1.0-rc change".
arcbbb edited the summary of this revision.
arcbbb added a comment.
keep Zvamo0p10 and remove the implicati
arcbbb updated this revision to Diff 356369.
arcbbb added a comment.
Fix FileCheck prefix: ZVAMO
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105396/new/
https://reviews.llvm.org/D105396
Files:
clang/lib/Driver/ToolChains/Arch/RISCV.cpp
clang
This revision was automatically updated to reflect the committed changes.
Closed by commit rGf1cbea3e5275: [RISCV] Remove Zvamo implication for v1.0-rc
change (authored by arcbbb).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105396/new/
https://re
arcbbb created this revision.
arcbbb added reviewers: craig.topper, rogfer01, frasercrmck, kito-cheng,
monkchiang, eopXD, khchen.
Herald added subscribers: VincentWu, luke957, achieveartificialintelligence,
vkmr, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna,
psnobl, jocewe
arcbbb added inline comments.
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:4683
+Ops.push_back(VL);
+Ops.push_back(DAG.getUNDEF(XLenVT)); // Policy
+ }
kito-cheng wrote:
> Is this operand for tail policy? if so why this is `UNDEF`? I guess thi
arcbbb accepted this revision.
arcbbb added a comment.
This revision is now accepted and ready to land.
LGTM too. Thanks.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115709/new/
https://reviews.llvm.org/D115709
__
arcbbb created this revision.
arcbbb added reviewers: craig.topper, rogfer01, khchen, HsiangKai, evandro,
liaolucy, jrtc27.
Herald added subscribers: vkmr, frasercrmck, dexonsmith, luismarques, apazos,
sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o,
brucehoult, MartinMosbeck
arcbbb created this revision.
arcbbb added reviewers: asb, craig.topper, frasercrmck, rogfer01, jrtc27,
mgrang.
Herald added subscribers: vkmr, evandro, luismarques, apazos, sameer.abuasal,
s.egerton, Jim, benna, psnobl, abidh, jocewei, PkmX, the_o, brucehoult,
MartinMosbeck, edward-jones, zzhen
arcbbb updated this revision to Diff 338004.
arcbbb added a comment.
Addressed @MaskRay 's comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100615/new/
https://reviews.llvm.org/D100615
Files:
clang/lib/Driver/ToolChains/RISCVToolchain.cpp
This revision was automatically updated to reflect the committed changes.
Closed by commit rG27edaee84e3e: [RISCV][Driver] Make the ordering of CmdArgs
consistent between RISCV::Linker… (authored by arcbbb).
Changed prior to commit:
https://reviews.llvm.org/D100615?vs=338004&id=338410#toc
Repo
arcbbb updated this revision to Diff 338416.
arcbbb marked 3 inline comments as done.
arcbbb added a comment.
Addressed @khchen's comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100448/new/
https://reviews.llvm.org/D100448
Files:
clang/in
arcbbb updated this revision to Diff 338419.
arcbbb added a comment.
re-formatted.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100448/new/
https://reviews.llvm.org/D100448
Files:
clang/include/clang/Basic/riscv_vector.td
clang/test/CodeGen/R
This revision was automatically updated to reflect the committed changes.
Closed by commit rG11072a0bdbc0: [RISCV][Clang] Add RVV AMO builtins (authored
by arcbbb).
Changed prior to commit:
https://reviews.llvm.org/D100448?vs=338419&id=339139#toc
Repository:
rG LLVM Github Monorepo
CHANGES
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
___
19 matches
Mail list logo
