https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #14 from vries at gcc dot gnu.org ---
Author: vries
Date: Tue Apr 28 06:05:14 2015
New Revision: 222509
URL: https://gcc.gnu.org/viewcvs?rev=222509root=gccview=rev
Log:
Evaluate side-effects in expand_ifn_va_arg_1
2015-04-28 Tom de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
vries at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #13 from vries at gcc dot gnu.org ---
*** Bug 65824 has been marked as a duplicate of this bug. ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
vries at gcc dot gnu.org changed:
What|Removed |Added
Keywords||patch
--- Comment #12 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #9 from dave.anglin at bell dot net ---
On 2015-04-22, at 12:40 PM, vries at gcc dot gnu.org wrote:
this patch is more conservative:
...
diff --git a/gcc/gimplify.c b/gcc/gimplify.c
index 7786e16..e23f510 100644
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #10 from vries at gcc dot gnu.org ---
(In reply to dave.anglin from comment #9)
This patch has the following new failure:
/test/gnu/gcc/gcc/gcc/testsuite/g++.dg/pr65802.C: In function 'void fn1()':
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #11 from vries at gcc dot gnu.org ---
Bootstrap and reg-test on x86_64 also went ok for for the updated patch. I'll
submit.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #7 from vries at gcc dot gnu.org ---
Hmm, this patch has failures for x86_64:
- va_arg (ap, void):
gcc.c-torture/compile/pr48767.c
- vla:
gcc.c-torture/execute/20020412-1.c
gcc.dg/compat/struct-by-value-22
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #3 from vries at gcc dot gnu.org ---
The code snippet:
...
case 'f':
case 'e':
case 'E':
case 'g':
case 'G':
(void) __builtin_va_arg(ap, double);
total_width += 307;
break;
...
translates
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #8 from dave.anglin at bell dot net ---
On 2015-04-22 12:40 PM, vries at gcc dot gnu.org wrote:
this patch is more conservative
Restarted test with updated patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
vries at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #5 from vries at gcc dot gnu.org ---
Dave,
could you please test the patch on hppa?
Thanks,
- Tom
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
--- Comment #6 from dave.anglin at bell dot net ---
On 2015-04-22, at 3:15 AM, vries at gcc dot gnu.org wrote:
could you please test the patch on hppa?
Started. Thanks for the patch.
Dave
--
John David Anglindave.ang...@bell.net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
vries at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65818
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Keywords||build
15 matches
Mail list logo