public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
@ 2012-05-02 19:08 meissner at gcc dot gnu.org
  2012-05-02 19:16 ` [Bug target/53199] " meissner at gcc dot gnu.org
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-02 19:08 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

             Bug #: 53199
           Summary: __builtin_bswap64 and __builtin_bswap32 generate
                    errors if -mcpu=power6
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: meissner@gcc.gnu.org
        ReportedBy: meissner@gcc.gnu.org
              Host: powerpc64-linux power-linux
            Target: powerpc64-linux
             Build: powerpc64-linux


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
@ 2012-05-02 19:16 ` meissner at gcc dot gnu.org
  2012-05-03 21:48 ` meissner at gcc dot gnu.org
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-02 19:16 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #1 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-02 19:16:26 UTC ---
If you compile the following with -mcpu=power6 -m64:

long long
b64 (long long *p)
{
  return __builtin_bswap64 (*p);
}

Using the -mno-avoid-indexed-addresses works around the problem.  Setting
cpu=power5 works because -mcpu=power5 does not set -mavoid-indexed-addresses by
default.  Setting -mcpu=power7 works because the power7 has a 64-bit load and
reverse bytes instruction.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
  2012-05-02 19:16 ` [Bug target/53199] " meissner at gcc dot gnu.org
@ 2012-05-03 21:48 ` meissner at gcc dot gnu.org
  2012-05-04  0:28 ` meissner at gcc dot gnu.org
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-03 21:48 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #2 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-03 21:47:49 UTC ---
Author: meissner
Date: Thu May  3 21:47:45 2012
New Revision: 187119

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187119
Log:
[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.


Added:
    trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/rs6000/rs6000.md
    trunk/gcc/testsuite/ChangeLog


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
  2012-05-02 19:16 ` [Bug target/53199] " meissner at gcc dot gnu.org
  2012-05-03 21:48 ` meissner at gcc dot gnu.org
@ 2012-05-04  0:28 ` meissner at gcc dot gnu.org
  2012-05-04  0:32 ` meissner at gcc dot gnu.org
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-04  0:28 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #3 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-04 00:28:21 UTC ---
Author: meissner
Date: Fri May  4 00:28:17 2012
New Revision: 187132

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187132
Log:
[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.


Added:
    branches/gcc-4_7-branch/gcc/testsuite/gcc.target/powerpc/pr53199.c
      - copied unchanged from r187119,
trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c
Modified:
    branches/gcc-4_7-branch/gcc/ChangeLog
    branches/gcc-4_7-branch/gcc/config/rs6000/rs6000.md   (contents, props
changed)
    branches/gcc-4_7-branch/gcc/testsuite/ChangeLog

Propchange: branches/gcc-4_7-branch/gcc/config/rs6000/rs6000.md
            ('svn:mergeinfo' added)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2012-05-04  0:28 ` meissner at gcc dot gnu.org
@ 2012-05-04  0:32 ` meissner at gcc dot gnu.org
  2012-05-04  1:07 ` meissner at gcc dot gnu.org
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-04  0:32 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #4 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-04 00:31:53 UTC ---
Author: meissner
Date: Fri May  4 00:31:50 2012
New Revision: 187133

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187133
Log:
[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.


Added:
    branches/gcc-4_6-branch/gcc/testsuite/gcc.target/powerpc/pr53199.c
      - copied unchanged from r187119,
trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c
Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/config/rs6000/rs6000.md   (contents, props
changed)
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog

Propchange: branches/gcc-4_6-branch/gcc/config/rs6000/rs6000.md
            ('svn:mergeinfo' added)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2012-05-04  0:32 ` meissner at gcc dot gnu.org
@ 2012-05-04  1:07 ` meissner at gcc dot gnu.org
  2012-05-04  1:46 ` meissner at gcc dot gnu.org
  2014-12-05 17:51 ` meissner at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-04  1:07 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #5 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-04 01:07:28 UTC ---
Author: meissner
Date: Fri May  4 01:07:24 2012
New Revision: 187137

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=187137
Log:
[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.


Added:
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/powerpc/pr53199.c
      - copied unchanged from r187119,
trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/config/rs6000/rs6000.md   (contents, props
changed)
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog

Propchange: branches/gcc-4_5-branch/gcc/config/rs6000/rs6000.md
            ('svn:mergeinfo' added)


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2012-05-04  1:07 ` meissner at gcc dot gnu.org
@ 2012-05-04  1:46 ` meissner at gcc dot gnu.org
  2014-12-05 17:51 ` meissner at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2012-05-04  1:46 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

Michael Meissner <meissner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #6 from Michael Meissner <meissner at gcc dot gnu.org> 2012-05-04 01:45:40 UTC ---
Fixed in subversion revision 187119, and back ported to 4.5, 4.6, and 4.7
branches.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/53199] __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6
  2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2012-05-04  1:46 ` meissner at gcc dot gnu.org
@ 2014-12-05 17:51 ` meissner at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: meissner at gcc dot gnu.org @ 2014-12-05 17:51 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53199

--- Comment #7 from Michael Meissner <meissner at gcc dot gnu.org> ---
Author: meissner
Date: Fri Dec  5 17:50:40 2014
New Revision: 218431

URL: https://gcc.gnu.org/viewcvs?rev=218431&root=gcc&view=rev
Log:
2014-12-05  Michael Meissner  <meissner@linux.vnet.ibm.com>

    PR target/53199
    * gcc.target/powerpc/pr53199.c: Limit tests to 64-bit.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2014-12-05 17:51 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-02 19:08 [Bug target/53199] New: __builtin_bswap64 and __builtin_bswap32 generate errors if -mcpu=power6 meissner at gcc dot gnu.org
2012-05-02 19:16 ` [Bug target/53199] " meissner at gcc dot gnu.org
2012-05-03 21:48 ` meissner at gcc dot gnu.org
2012-05-04  0:28 ` meissner at gcc dot gnu.org
2012-05-04  0:32 ` meissner at gcc dot gnu.org
2012-05-04  1:07 ` meissner at gcc dot gnu.org
2012-05-04  1:46 ` meissner at gcc dot gnu.org
2014-12-05 17:51 ` meissner at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).