Davide Italiano via llvm-dev
2017-Sep-28 20:35 UTC
[llvm-dev] [RFC] Adding a cls intrinsic for AArch64
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. Thanks, -- Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare
Davide Italiano via llvm-dev
2017-Sep-28 20:37 UTC
[llvm-dev] [RFC] Adding a cls intrinsic for AArch64
On Thu, Sep 28, 2017 at 1:35 PM, Davide Italiano <davide at freebsd.org> wrote:> 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.I wonder *if*, sigh.
Davide Italiano via llvm-dev
2017-Oct-04 17:18 UTC
[llvm-dev] [RFC] Adding a cls intrinsic for AArch64
On Thu, Sep 28, 2017 at 1:35 PM, Davide Italiano <davide at freebsd.org> wrote:> 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. > > Thanks, > > -- > DavideNo comments, I'll take that as a sign I can go ahead :) -- Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare
Eric Christopher via llvm-dev
2017-Oct-05 18:46 UTC
[llvm-dev] [RFC] Adding a cls intrinsic for AArch64
On Wed, Oct 4, 2017 at 10:19 AM Davide Italiano via llvm-dev < llvm-dev at lists.llvm.org> wrote:> On Thu, Sep 28, 2017 at 1:35 PM, Davide Italiano <davide at freebsd.org> > wrote: > > 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. :) -eric> > Thanks, > > > > -- > > Davide > > No comments, I'll take that as a sign I can go ahead :) > > -- > Davide > > "There are no solved problems; there are only problems that are more > or less solved" -- Henri Poincare > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20171005/491cb992/attachment-0001.html>