[Clang][SVE2p1] Add svpsel builtins

[CarolineConcatto]

Summary

As described in: https://github.com/ARM-software/acle/pull/257

Patch by : Sander de Smalen<[e-mail]>

[hassnaa-arm]

clang/include/clang/Basic/arm_sve.td

2123

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 ?

clang/lib/CodeGen/CGBuiltin.cpp

9633–9635

for the case of sve::BI__builtin_sve_svpsel_lane_b8,
what is the expected value of IsSVCount ? and how the assertion statement didn't assert for the check of :
(cast<TargetExtType>(Ops[0]->getType())->getName() ==

"aarch64.svcount"))

how is the parameter type considered as aarch64.svcount ?

9642

Isn't the type of 'SVCountTy' = 'aarch64.svcount' as a result of the statement at line 9637 ?
So why do we need to aarch64_sve_convert_from_svbool while it's not svbool.
Am I missing something ?

[hassnaa-arm]

clang/lib/CodeGen/CGBuiltin.cpp

9633–9635

Hi Carol,
I understood that part, ignore my comment.

9642

I understood that part, please ignore my comment.