https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #10 from Jakub Jelinek ---
Author: jakub
Date: Wed Jan 21 21:23:04 2015
New Revision: 219969
URL: https://gcc.gnu.org/viewcvs?rev=219969&root=gcc&view=rev
Log:
PR c/63307
* cilk.c (fill_decls_vec): Only put decls into vector
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
Jakub Jelinek changed:
What|Removed |Added
Target Milestone|4.9.2 |4.9.3
--- Comment #9 from Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
Tobias Burnus changed:
What|Removed |Added
CC||burnus at gcc dot gnu.org
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #7 from Andrew Pinski ---
(In reply to iverbin from comment #6)
> Author: iverbin
> Date: Mon Oct 20 15:22:09 2014
> New Revision: 216483
This breaks the build as wd->decl_map will always contain a BLOCK which does
not have an UID.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #6 from iverbin at gcc dot gnu.org ---
Author: iverbin
Date: Mon Oct 20 15:22:09 2014
New Revision: 216483
URL: https://gcc.gnu.org/viewcvs?rev=216483&root=gcc&view=rev
Log:
PR c/63307
gcc/c-family/
* cilk.c: Include vec.h.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #5 from Igor Zamyatin ---
(In reply to Jakub Jelinek from comment #4)
> I don't think so. They copy declarations, i.e. create new declarations, and
> the different ordering of their DECL_UID values may result in code
> generation dif
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #3 from Igor Zamyatin ---
(In reply to Jakub Jelinek from comment #2)
>
> > + vec_arglist.release();
>
> Formatting. You could use auto_vec, perhaps with some stack allocated
> initial buffer if you think say 16 vector elements w
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #4 from Jakub Jelinek ---
(In reply to Igor Zamyatin from comment #3)
> (In reply to Jakub Jelinek from comment #2)
>
> >
> > > + vec_arglist.release();
> >
> > Formatting. You could use auto_vec, perhaps with some stack allocate
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #2 from Jakub Jelinek ---
(In reply to Igor Zamyatin from comment #1)
> Would like to ask here first - will something like following be ok:
> +typedef struct
> +{
> + tree parm;
> + tree arg;
> +} decl_pair;
> +
> +static vec vec_a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
--- Comment #1 from Igor Zamyatin ---
Would like to ask here first - will something like following be ok:
diff --git a/gcc/c-family/cilk.c b/gcc/c-family/cilk.c
index bf549ad..f453bc5 100644
--- a/gcc/c-family/cilk.c
+++ b/gcc/c-family/cilk.c
@
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63307
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |4.9.2
12 matches
Mail list logo
