This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, powerpc] PR 53199, fix usage of __builtin_bswap64 on power6
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, gcc-patches at gcc dot gnu dot org, Peter Bergner <bergner at vnet dot ibm dot com>
- Date: Thu, 3 May 2012 17:04:10 -0400
- Subject: Re: [PATCH, powerpc] PR 53199, fix usage of __builtin_bswap64 on power6
- References: <20120503175649.GA31666@ibm-tiger.the-meissners.org>
On Thu, May 3, 2012 at 1:56 PM, Michael Meissner
<meissner@linux.vnet.ibm.com> wrote:
> In trying to build machine specific versions of glibc with newer compilers, we
> ran into a bug where the current glibc would not build on power6. ?This was due
> to glibc using the __builtin_swap64 builtin. ?Power6 sets the
> -mavoid-indexed-addresses option. ?The splitter that I wrote when I wrote the
> bswap64 did not have an alternate code path for -mavoid-indexed-addresses.
> This patch adds the alternate code path. ?I bootstraped it with today's
> compiler and there were no regressions in make check. ?Is it ok to apply?
>
> [gcc]
> 2012-05-03 ?Michael Meissner ?<meissner@linux.vnet.ibm.com>
>
> ? ? ? ?PR target/53199
> ? ? ? ?* config/rs6000/rs6000.md (bswapdi splitters): If
> ? ? ? ?-mavoid-indexed-addresses (or -mcpu=power6 which sets it by
> ? ? ? ?default) is used, generate an alternate sequence that does not
> ? ? ? ?depend on using indexed addressing.
>
> [gcc/testsuite]
> 2012-05-03 ?Michael Meissner ?<meissner@linux.vnet.ibm.com>
>
> ? ? ? ?PR target/53199
> ? ? ? ?* gcc.target/powwerpc/pr53199.c: New file.
Okay.
Thanks, David