[RFC] Adding a cls intrinsic for AArch64

Project Infrastructure LLVM Dev List Archives
#1

I'm not entirely sure whether this is worth an RFC, but, just in case.
On bugzilla Eli pointed out we currently match a specific sequence of
instructions to cls (
rGb9a0551862fb), and given how fragile this is, it
could be better to just introduce an intrinsic for it (and presumably,
teach peephole optimizations to lower a sequence to that intrinsic).
I wonder what folks are OK with the new intrinsic.
If there are no strong objections, I'll proceed with a patch.

Thanks,

#2

I wonder *if*, sigh.

#3

No comments, I'll take that as a sign I can go ahead :slight_smile:

#4

I’m not entirely sure whether this is worth an RFC, but, just in case.
On bugzilla Eli pointed out we currently match a specific sequence of
instructions to cls (
https://reviews.llvm.org/rL206079), and given how fragile this is, it
could be better to just introduce an intrinsic for it (and presumably,
teach peephole optimizations to lower a sequence to that intrinsic).
I wonder what folks are OK with the new intrinsic.
If there are no strong objections, I’ll proceed with a patch.

No strong objections. :slight_smile:

-eric