https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #11 from Jakub Jelinek ---
Author: jakub
Date: Thu Feb 26 21:01:59 2015
New Revision: 221033
URL: https://gcc.gnu.org/viewcvs?rev=221033&root=gcc&view=rev
Log:
PR tree-optimization/65215
* tree-ssa-math-opts.c (find_bswap_or_
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #10 from Richard Biener ---
(In reply to Richard Biener from comment #9)
> (In reply to Thomas Preud'homme from comment #5)
> > (In reply to Richard Biener from comment #4)
> > > (In reply to Jakub Jelinek from comment #1)
> > > > Cre
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #9 from Richard Biener ---
(In reply to Thomas Preud'homme from comment #5)
> (In reply to Richard Biener from comment #4)
> > (In reply to Jakub Jelinek from comment #1)
> > > Created attachment 34879 [details]
> > > gcc5-pr65215.pat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #8 from Jakub Jelinek ---
(In reply to Thomas Preud'homme from comment #7)
> (In reply to Jakub Jelinek from comment #6)
> > I can certainly remove that hunk from the patch, if the expander and other
> > passes handle it well. The te
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #7 from Thomas Preud'homme ---
(In reply to Jakub Jelinek from comment #6)
> I can certainly remove that hunk from the patch, if the expander and other
> passes handle it well. The test can stay I guess.
Things are at least working
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #6 from Jakub Jelinek ---
I can certainly remove that hunk from the patch, if the expander and other
passes handle it well. The test can stay I guess.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #5 from Thomas Preud'homme ---
(In reply to Richard Biener from comment #4)
> (In reply to Jakub Jelinek from comment #1)
> > Created attachment 34879 [details]
> > gcc5-pr65215.patch
> >
> > Untested fix. There are still issues lef
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #4 from Richard Biener ---
(In reply to Jakub Jelinek from comment #1)
> Created attachment 34879 [details]
> gcc5-pr65215.patch
>
> Untested fix. There are still issues left, e.g. I can't understand the
> "bswap &&" part in
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
Jakub Jelinek changed:
What|Removed |Added
Attachment #34879|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #2 from Jakub Jelinek ---
I mean something like:
unsigned int
foo (unsigned char *p)
{
return ((unsigned int) p[0] << 24) | (p[1] << 16) | (p[2] << 8) | p[3];
}
on strict align big endian machines, let me add another testcase.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #1 from Jakub Jelinek ---
Created attachment 34879
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34879&action=edit
gcc5-pr65215.patch
Untested fix. There are still issues left, e.g. I can't understand the "bswap
&&" part in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
Jakub Jelinek changed:
What|Removed |Added
Priority|P3 |P1
Status|UNCONFIRMED
13 matches
Mail list logo
