https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
H.J. Lu changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #14 from CVS Commits ---
The master branch has been updated by H.J. Lu :
https://gcc.gnu.org/g:1f94ed3b4c308c9da7baf59ecbc0f953e994f9c4
commit r12-664-g1f94ed3b4c308c9da7baf59ecbc0f953e994f9c4
Author: H.J. Lu
Date: Sat May 8 07:2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #13 from andi at firstfloor dot org ---
> The code in the initial report optimizes to bswap with GCC8.1 and later.
> Is that the test case you meant? GCC8.1 was released on May 2, 2018, well
> before your Nov comment, so maybe you mea
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
Peter Cordes changed:
What|Removed |Added
CC||peter at cordes dot ca
--- Comment #12 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #11 from Andi Kleen ---
Only when the first test case is fixed too
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
Martin Liška changed:
What|Removed |Added
CC||marxin at gcc dot gnu.org
--- Comment #10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #9 from Andreas Schwab ---
The test fails on powerpc, both -m32 and -m64.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #8 from Richard Biener ---
Author: rguenth
Date: Fri May 13 12:27:18 2016
New Revision: 236204
URL: https://gcc.gnu.org/viewcvs?rev=236204&root=gcc&view=rev
Log:
2016-05-13 Richard Biener
PR tree-optimization/42587
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
--- Comment #7 from Richard Biener ---
I have a patch to fix the second testcase in comment#3. Note that the original
testcase still fails to be detected.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42587
xiaoyuanbo changed:
What|Removed |Added
CC||xiaoyuanbo at yeah dot net
--- Comment #6 fr
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-02-27 20:04 ---
*** Bug 43197 has been marked as a duplicate of this bug. ***
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #4 from jamborm at gcc dot gnu dot org 2010-01-06 14:07 ---
(In reply to comment #2)
> SRA here again does somthing stupid:
>
As I have already told richi on IRC, if "in.value = value_1(D)" was an
assignment of two unions, it would have loaded the chars from the rhs
and thu
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-01-03 16:26 ---
Similar example not handled by bswap recognition:
typedef unsigned char u8;
typedef unsigned int u32;
union __anonunion
{
u32 value;
u8 bytes[4];
};
u32
acpi_ut_dword_byte_swap (u32 value)
{
union __anonunion
--- Comment #2 from rguenth at gcc dot gnu dot org 2010-01-03 16:01 ---
SRA here again does somthing stupid:
from
:
acpi_ut_track_stack_ptr ();
in.value = value_1(D);
D.1966_2 = in.bytes[3];
out.bytes[0] = D.1966_2;
D.1967_3 = in.bytes[2];
out.bytes[1] = D.1967_3;
D.1968_
--- Comment #1 from rguenth at gcc dot gnu dot org 2010-01-03 10:58 ---
Confirmed.
The bswap detection does not work on memory at all but only detects it
on registers.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
15 matches
Mail list logo
