https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Rainer Orth changed:
What|Removed |Added
Resolution|--- |FIXED
Status|REOPENED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #21 from Jakub Jelinek ---
Yeah, but clearly not in libgomp, but in the testcase, so IMNSHO we shouldn't
reuse this PR for that. On x86_64-linux I see movaps into and out from the
thr.1 TLS variable and in two spots to/from stack, th
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Rainer Orth changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #18 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:17da2c7425ea1f5bf417b954f444dbe1f1618a1c
commit r12-5362-g17da2c7425ea1f5bf417b954f444dbe1f1618a1c
Author: Jakub Jelinek
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #17 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:7a2aa63fad06a72d9770b08491f1a7809eac7c50
commit r12-5361-g7a2aa63fad06a72d9770b08491f1a7809eac7c50
Author: Jakub Jelinek
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #16 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #15 from Jakub Jelinek ---
> Ah, bet Solaris aligned_alloc relies on:
> "the value of size shall be an integral multiple of alignment"
> (glibc aligned_alloc doesn't).
I
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #15 from Jakub Jelinek ---
Ah, bet Solaris aligned_alloc relies on:
"the value of size shall be an integral multiple of alignment"
(glibc aligned_alloc doesn't).
Does memalign or posix_memalign rely on that too, or just aligned_alloc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #14 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #13 from Jakub Jelinek ---
> Created attachment 51807
> --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51807&action=edit
> gcc12-pr102838-2.patch
>
> Does this patc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #13 from Jakub Jelinek ---
Created attachment 51807
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51807&action=edit
gcc12-pr102838-2.patch
Does this patch fix it?
Sorry for the delay...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Thomas Schwinge changed:
What|Removed |Added
CC||jules at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #11 from Jakub Jelinek ---
Ah, I see, there is still the team->work_shares array where the whole team
structure in which it is present is allocated with gomp_malloc.
So, either we need to drop the aligned (64) attribute regardless of
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #10 from Rainer Orth ---
Created attachment 51644
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51644&action=edit
32-bit i386-pc-solaris2.11 loop_ull.[is]
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #9 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #8 from Jakub Jelinek ---
> Ok, so, first question, is GOMP_HAVE_EFFICIENT_ALIGNED_ALLOC defined in your
> case?
It is since all of HAVE_ALIGNED_ALLOC, HAVE_POSIX_MEMALIG
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #8 from Jakub Jelinek ---
Ok, so, first question, is GOMP_HAVE_EFFICIENT_ALIGNED_ALLOC defined in your
case?
Can be seen e.g. from objdump -dr alloc.o and seeing if gomp_aligned_free is
just a (tail?) call to free without anything el
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #7 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #6 from ro at CeBiTec dot Uni-Bielefeld.DE Uni-Bielefeld.DE> ---
>> --- Comment #5 from Jakub Jelinek ---
>> Does the committed patch fix the issue on Solaris?
>
> I'll s
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #6 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #5 from Jakub Jelinek ---
> Does the committed patch fix the issue on Solaris?
I'll see after tonight's bootstrap. The original one attached to the PR
fixed only a few o
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #5 from Jakub Jelinek ---
Does the committed patch fix the issue on Solaris?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #4 from CVS Commits ---
The master branch has been updated by Jakub Jelinek :
https://gcc.gnu.org/g:c7abdf46fb7ac9a0c37f120feff3fcc3a752584f
commit r12-4529-gc7abdf46fb7ac9a0c37f120feff3fcc3a752584f
Author: Jakub Jelinek
Date: W
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Jakub Jelinek changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org
Last
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #2 from Jakub Jelinek ---
Actually no, I'm wrong, there is
gomp_mutex_t lock __attribute__((aligned (64)));
in the middle of the structure, so the start of the structure is 64-byte
aligned.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
--- Comment #1 from Jakub Jelinek ---
Possibly triggered by enabling vectorization at -O2?
long long should for ia32 inside of structures (at least on linux but I think
it is the same for solaris too) have just 32-bit alignment and nothing else
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102838
Rainer Orth changed:
What|Removed |Added
Target Milestone|--- |12.0
23 matches
Mail list logo
