https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #36 from Eric Botcazou ---
> Don't be silly. I've fixed all regresseions that have shown up, except the
> one on Sparc because you refused to accept changes to the Sparc backend.
Well, this is somewhat contradictory with your
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #35 from Dominik Vogt ---
(In reply to Eric Botcazou from comment #34)
> > I still disagree with reverting the patch. There was plenty of time to
> > identify and fix affected backends instead of doing nothing for half five
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #34 from Eric Botcazou ---
> I still disagree with reverting the patch. There was plenty of time to
> identify and fix affected backends instead of doing nothing for half five
> months and then claiming that the patch is potentially
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #33 from Dominik Vogt ---
I still disagree with reverting the patch. There was plenty of time to
identify and fix affected backends instead of doing nothing for half five
months and then claiming that the patch is potentially too
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
Eric Botcazou changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #31 from Eric Botcazou ---
Author: ebotcazou
Date: Thu Feb 2 12:39:09 2017
New Revision: 245124
URL: https://gcc.gnu.org/viewcvs?rev=245124=gcc=rev
Log:
PR middle-end/78468
* emit-rtl.c (init_emit): Add ??? comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #30 from Dominik Vogt ---
(In reply to Eric Botcazou from comment #24)
> The root cause of this mess is actually init_emit:
>
> REGNO_POINTER_ALIGN (VIRTUAL_INCOMING_ARGS_REGNUM) = STACK_BOUNDARY;
> REGNO_POINTER_ALIGN
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #29 from Jakub Jelinek ---
(In reply to Eric Botcazou from comment #28)
> > Well, at least at this point in GCC 7 development the fewer architectures we
> > change the better.
>
> Sure, but by changing the middle-end, you change all
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #28 from Eric Botcazou ---
> Well, at least at this point in GCC 7 development the fewer architectures we
> change the better.
Sure, but by changing the middle-end, you change all the architectures at once
and may break a fair
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #27 from Jakub Jelinek ---
(In reply to Eric Botcazou from comment #26)
> > So do we want a target hook that can override these alignments, or new
> > macros for each of the alignments that default to STACK_BOUNDARY?
>
> No strong
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #26 from Eric Botcazou ---
> So do we want a target hook that can override these alignments, or new
> macros for each of the alignments that default to STACK_BOUNDARY?
No strong opinion, but IMO the main issue is to arrange for the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #25 from Jakub Jelinek ---
So do we want a target hook that can override these alignments, or new macros
for each of the alignments that default to STACK_BOUNDARY?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #24 from Eric Botcazou ---
The root cause of this mess is actually init_emit:
REGNO_POINTER_ALIGN (VIRTUAL_INCOMING_ARGS_REGNUM) = STACK_BOUNDARY;
REGNO_POINTER_ALIGN (VIRTUAL_STACK_VARS_REGNUM) = STACK_BOUNDARY;
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
--- Comment #23 from Eric Botcazou ---
Created attachment 40597
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40597=edit
Tentative fix I'm going to test.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
Dominik Vogt changed:
What|Removed |Added
CC||vogt at linux dot vnet.ibm.com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78468
Eric Botcazou changed:
What|Removed |Added
Component|libgomp |middle-end
Severity|normal
17 matches
Mail list logo
