efriedma added inline comments.
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7683
+if (TypeFlags.isExpandOp1SVALL()) {
+ if (Ops.size() <= 1)
+Ops.push_back(Builder.getInt32(31));
sdesmalen wrote:
> efriedma wrote:
> > The `Ops.size() <= 1` seems
sdesmalen marked an inline comment as done.
sdesmalen added inline comments.
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7683
+if (TypeFlags.isExpandOp1SVALL()) {
+ if (Ops.size() <= 1)
+Ops.push_back(Builder.getInt32(31));
efriedma wrote:
> The
efriedma added inline comments.
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7683
+if (TypeFlags.isExpandOp1SVALL()) {
+ if (Ops.size() <= 1)
+Ops.push_back(Builder.getInt32(31));
The `Ops.size() <= 1` seems to return the same result for all the i
sdesmalen updated this revision to Diff 257834.
sdesmalen added a comment.
rebased patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77597/new/
https://reviews.llvm.org/D77597
Files:
clang/include/clang/Basic/TargetBuiltins.h
clang/include/clang/Basic/arm_sve.td
clang/lib/Code
sdesmalen created this revision.
sdesmalen added reviewers: SjoerdMeijer, efriedma, rovka.
Herald added a subscriber: tschuett.
Herald added a reviewer: rengolin.
Herald added a project: clang.
sdesmalen added a parent revision: D77596: [SveEmitter] Add NoOverload flag and
builtin for svpfalse.
S
