https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #16 from Martin Liška ---
Author: marxin
Date: Mon Apr 15 08:53:24 2019
New Revision: 270365
URL: https://gcc.gnu.org/viewcvs?rev=270365&root=gcc&view=rev
Log:
Backport r268060
2019-04-15 Martin Liska
Backport from mainl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
Status|RESOLVED|NEW
Known to work|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
CC||nheart at gmail dot com
--- Comment #15 f
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #14 from hjl at gcc dot gnu.org ---
Author: hjl
Date: Fri Jan 18 14:33:46 2019
New Revision: 268079
URL: https://gcc.gnu.org/viewcvs?rev=268079&root=gcc&view=rev
Log:
Update PR middle-end/88587 tests
It is wrong to use -m32 in dg-op
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #12 from Martin Liška ---
Author: marxin
Date: Fri Jan 18 07:41:05 2019
New Revision: 268060
URL: https://gcc.gnu.org/viewcvs?rev=268060&root=gcc&view=rev
Log:
Reset proper type on vector types (PR middle-end/88587).
2019-01-18 Mar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #11 from Richard Biener ---
Yes, it depends on the supported ISAs. In essential the issue is that
TYPE_MODE
depends on the (function) context it is used, that's even true for globals
(IIRC there are existing bugs with respect to that
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #10 from Martin Liška ---
(In reply to Martin Liška from comment #9)
> > I think the canonical way of re-computing DECL_MODE would be to re-layout
> > decls since that also updates RTL if that was already set. There's
> > relayout_de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #9 from Martin Liška ---
> I think the canonical way of re-computing DECL_MODE would be to re-layout
> decls since that also updates RTL if that was already set. There's
> relayout_decl for this which is for example called from omp-s
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #8 from Martin Liška ---
> It's quite strange that following is fine:
>
> $ cat pr88587-2.c
> __attribute__((target("default"))) void a() {
> __attribute__((__vector_size__(4 * sizeof(float int b = {};
> }
>
> __attribute__((t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #7 from Richard Biener ---
(In reply to Jakub Jelinek from comment #5)
> I think the problem is that while we make TYPE_MODE something dynamic for
> vector types (which takes into account whether the current function supports
> in hw
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #6 from Jakub Jelinek ---
Or just recompute DECL_MODE on all decls with vector type in the function
during the cloning for a different ISA.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #5 from Jakub Jelinek ---
I think the problem is that while we make TYPE_MODE something dynamic for
vector types (which takes into account whether the current function supports in
hw such vectors or not):
#define TYPE_MODE(NODE) \
(
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
--- Comment #3 from Martin Liška ---
So the problem is that new function body is copied in:
(gdb) bt
#0 __memmove_avx_unaligned_erms () at
../sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S:342
#1 0x00fd7378 in copy_node (node=no
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88587
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
18 matches
Mail list logo