https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #16 from Jakub Jelinek ---
Author: jakub
Date: Thu Dec 1 23:15:57 2016
New Revision: 243145
URL: https://gcc.gnu.org/viewcvs?rev=243145=gcc=rev
Log:
PR tree-optimization/78586
* gimple-ssa-sprintf.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
Jakub Jelinek changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #14 from Martin Sebor ---
(In reply to Jakub Jelinek from comment #13)
> (In reply to Martin Sebor from comment #11)
> > PR tree-optimization/78586
> > * gcc.dg/tree-ssa/builtin-sprintf-2.c: New test cases.
> >
> >
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #13 from Jakub Jelinek ---
(In reply to Martin Sebor from comment #11)
> PR tree-optimization/78586
> * gcc.dg/tree-ssa/builtin-sprintf-2.c: New test cases.
>
>
> Modified:
> trunk/gcc/testsuite/ChangeLog
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
Martin Sebor changed:
What|Removed |Added
See Also||https://gcc.gnu.org/bugzill
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #11 from Martin Sebor ---
Author: msebor
Date: Wed Nov 30 19:16:03 2016
New Revision: 243081
URL: https://gcc.gnu.org/viewcvs?rev=243081=gcc=rev
Log:
PR tree-optimization/78586 - [7 Regression] Wrong code caused by
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #10 from Martin Sebor ---
(In reply to Jakub Jelinek from comment #8)
> The far more important thing is that the handling of different argtype and
> dirtype really doesn't work. Consider:
>
> __attribute__((noinline, noclone)) int
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #9 from Jakub Jelinek ---
Even for TYPE_PRECISION (dirtype) == TYPE_PRECISION (argtype), if one is signed
and the other is unsigned, are you sure that for VR_RANGE we do the right
thing?
Again, should be backed out by sufficient
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #8 from Jakub Jelinek ---
Created attachment 40201
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40201=edit
gcc7-pr78586-2.patch
Ok, attaching a patch for the minor nits listed above.
The far more important thing is that the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #7 from Jakub Jelinek ---
Author: jakub
Date: Wed Nov 30 08:01:47 2016
New Revision: 242998
URL: https://gcc.gnu.org/viewcvs?rev=242998=gcc=rev
Log:
PR tree-optimization/78586
* gimple-ssa-sprintf.c (format_integer):
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #6 from Martin Sebor ---
(In reply to Jakub Jelinek from comment #5)
> There are still various weird spots in format_integer.
> E.g.
> else if (TREE_CODE (TREE_TYPE (arg)) == INTEGER_TYPE
>|| TREE_CODE (TREE_TYPE (arg))
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
Jakub Jelinek changed:
What|Removed |Added
CC||law at gcc dot gnu.org
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #4 from Jakub Jelinek ---
Created attachment 40190
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40190=edit
gcc7-pr78586.patch
Untested fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
--- Comment #3 from Jakub Jelinek ---
--- gcc/gimple-ssa-sprintf.c.jj 2016-11-28 23:50:20.0 +0100
+++ gcc/gimple-ssa-sprintf.c2016-11-29 15:03:29.201488577 +0100
@@ -1159,16 +1159,15 @@ format_integer (const conversion_spec
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78586
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
Status|UNCONFIRMED
16 matches
Mail list logo