https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #26 from Tom de Vries ---
Author: vries
Date: Sun Aug 28 05:05:15 2016
New Revision: 239805
URL: https://gcc.gnu.org/viewcvs?rev=239805&root=gcc&view=rev
Log:
Tag {ms,sysv}_va_list_type_node with '{ms,sysv}_abi va_list' attribute
20
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Tom de Vries changed:
What|Removed |Added
Keywords||patch
--- Comment #25 from Tom de Vries
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #24 from rguenther at suse dot de ---
On Tue, 23 Aug 2016, vries at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
>
> --- Comment #23 from Tom de Vries ---
> (In reply to Richard Biener from comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #23 from Tom de Vries ---
(In reply to Richard Biener from comment #22)
> (In reply to vries from comment #18)
> > Created attachment 39484 [details]
> > Tentative patch using attributes ms_abi/sysv_abi
> >
> > not build or regressio
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #22 from Richard Biener ---
(In reply to vries from comment #18)
> Created attachment 39484 [details]
> Tentative patch using attributes ms_abi/sysv_abi
>
> not build or regression tested yet, but at least makes test-case pass.
LGTM
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #21 from Steven Shi ---
Vries,
Sorry, my bad. I didn't apply your patch completely. Yes, your patch works.
And with your patch, my UEFU firmware gcc lto build pass and boot successfully.
Hope we could check in a formal fix ASAP, becau
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #20 from vries at gcc dot gnu.org ---
(In reply to Steven Shi from comment #19)
> (In reply to vries from comment #18)
> > Created attachment 39484 [details]
> > Tentative patch using attributes ms_abi/sysv_abi
> >
> > not build or re
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #19 from Steven Shi ---
(In reply to vries from comment #18)
> Created attachment 39484 [details]
> Tentative patch using attributes ms_abi/sysv_abi
>
> not build or regression tested yet, but at least makes test-case pass.
Vries,
I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #18 from vries at gcc dot gnu.org ---
Created attachment 39484
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39484&action=edit
Tentative patch using attributes ms_abi/sysv_abi
not build or regression tested yet, but at least ma
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #17 from Richard Biener ---
(In reply to Steven Shi from comment #16)
> How to checkout the 6.3 code from GCC trunk? I hope to verify this fix on
> Uefi firmware GCC LTO build.
The patch is not checked in as I believe the fundamental
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #16 from Steven Shi ---
How to checkout the 6.3 code from GCC trunk? I hope to verify this fix on Uefi
firmware GCC LTO build.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Richard Biener changed:
What|Removed |Added
CC||steven.shi at intel dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Richard Biener changed:
What|Removed |Added
Target Milestone|6.2 |6.3
--- Comment #13 from Richard Biener
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Richard Biener changed:
What|Removed |Added
Target Milestone|6.2 |6.3
--- Comment #13 from Richard Biener
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #12 from Richard Biener ---
The patch hints at type merging not being performed properly. AFAIR I saw
that for some of the eventually late built "global" types in the backend:
/* Calling abi specific va_list type nodes. */
static G
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #11 from Domani Hannes ---
I can confirm that this patch works for windows as well.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #10 from zenith432 at users dot sourceforge.net ---
(In reply to vries from comment #8)
> Created attachment 38453 [details]
> tentative patch
vries, thank you very much. I verified and looks good.
Built GCC 6.1.0 with patch from re
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Domani Hannes changed:
What|Removed |Added
CC||ssbssa at yahoo dot de
--- Comment #9 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
vries at gcc dot gnu.org changed:
What|Removed |Added
CC||vries at gcc dot gnu.org
--- C
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Richard Biener changed:
What|Removed |Added
Keywords||lto, wrong-code
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
--- Comment #6 from zenith432 at users dot sourceforge.net ---
(In reply to H.J. Lu from comment #5)
> This may be related to PR 50818.
PR 50818 has morphed into a feature request that the canonical
__builtin_va_{list,start,end,copy} builtins beh
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
H.J. Lu changed:
What|Removed |Added
CC||hjl.tools at gmail dot com
--- Comment #5 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70955
Dominique d'Humieres changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
23 matches
Mail list logo