As described in: https://github.com/ARM-software/acle/pull/257
Patch by : Sander de Smalen<sander.desmalen@arm.com>
Paths
| Differential D151197
[Clang][SVE2p1] Add svpsel builtins ClosedPublic Authored by CarolineConcatto on May 23 2023, 3:32 AM.
Details Summary As described in: https://github.com/ARM-software/acle/pull/257 Patch by : Sander de Smalen<sander.desmalen@arm.com>
Diff Detail
Event TimelineCarolineConcatto added a parent revision: D151081: [Clang][SVE2.1] Add svpext builtins.May 23 2023, 3:33 AM CarolineConcatto added a child revision: D151199: [Clang][SVE2.1] Add pfalse builtin.May 23 2023, 3:55 AM
Comment Actions Thank you for updating this @CarolineConcatto, LGTM
This revision is now accepted and ready to land.Oct 18 2023, 3:37 AM Closed by commit rG1b93e15bcd9a: [Clang][SVE2p1] Add svpsel builtins (authored by CarolineConcatto). · Explain WhyOct 18 2023, 8:05 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 557739 clang/include/clang/Basic/arm_sve.td
clang/lib/CodeGen/CGBuiltin.cpp
clang/test/CodeGen/aarch64-sve2p1-intrinsics/acle_sve2p1_psel.c
|
In the section of prototype modifiers: "}}Pmi"
According to the used modifiers, I see that they refer to 4 parameters, while in the testing file I see the function takes 3 parameters only.
Isn't that 'i' modifier extra ?
The same case for all the builtins.
Am I correct ?