This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386]: Put back __builtin_clzs and __builtin_ctzs
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 14 Dec 2016 16:33:55 +0100
- Subject: Re: [PATCH, i386]: Put back __builtin_clzs and __builtin_ctzs
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4aCMb62qDhLV8Std66An7CGQpA2PNwYx3GqWRxpEjZsdg@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Dec 14, 2016 at 04:30:39PM +0100, Uros Bizjak wrote:
> Apparently, Qt source uses these two builtins. As the removal brings
> much pain, I'd like to commit the following patch to restore
> __builtin_clzs and __builtin_ctzs on gcc-6 branch.
>
> 2016-12-14 Uros Bizjak <ubizjak@gmail.com>
>
> PR target/59874
> * config/i386/i386.c (enum ix86_builtins): Add IX86_BUILTIN_CLZS
> and IX86_BUILTIN_CTZS.
> (bdesc_args): Add __builtin_clzs and __builtin_ctzs.
> (ix86_fold_builtin): Handle IX86_BUILTIN_CTZS and IX86_BUILTIN_CLZS.
>
> Bootstrapped and regression tested on x86_64-linux-gnu.
Ok for 6. For 7 we ask them to change it to the new builtins?
Jakub