https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Mikael Morin changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #38 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:237380cdec2757bd42a0ec4d426b181f77d31d18
commit r15-5502-g237380cdec2757bd42a0ec4d426b181f77d31d18
Author: Mikael Morin
Date: Wed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #37 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:f5a87c8d8c6a8cfcd23595e67d3b86939e01c75c
commit r15-5477-gf5a87c8d8c6a8cfcd23595e67d3b86939e01c75c
Author: Mikael Morin
Date: Thu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #36 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:f74f52642fc0bd6b4c6828bd6e86aa5bb206cbca
commit r15-5463-gf74f52642fc0bd6b4c6828bd6e86aa5bb206cbca
Author: Mikael Morin
Date: Tue
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #34 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:8aa254d7a5db5189ccd944b6ca1f5443726b058d
commit r15-5446-g8aa254d7a5db5189ccd944b6ca1f5443726b058d
Author: Mikael Morin
Date: Thu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #33 from Tamar Christina ---
Many Thanks Mikael!
I see the functions being inlined now!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #31 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:3c01ddc4ff0fdbaf32c22aed1c04d1d587821d91
commit r15-3764-g3c01ddc4ff0fdbaf32c22aed1c04d1d587821d91
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #32 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:d6cb7794dcd7d97b77283fbbe0026e63521d2375
commit r15-3765-gd6cb7794dcd7d97b77283fbbe0026e63521d2375
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #29 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:5999d558e74b3531536c74976e8f05bb3ed31335
commit r15-3762-g5999d558e74b3531536c74976e8f05bb3ed31335
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #27 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:c313924c3bcb4fa672c7c1880fe686bd5af2cc7a
commit r15-3758-gc313924c3bcb4fa672c7c1880fe686bd5af2cc7a
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #30 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:7d43b4e06786c1023210f90e5231bde947aef3af
commit r15-3763-g7d43b4e06786c1023210f90e5231bde947aef3af
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #28 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:dd5250384dc91b4db117c599095f47955b6dc124
commit r15-3761-gdd5250384dc91b4db117c599095f47955b6dc124
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #25 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:b6ea4f7b0e6f983376aac6ebcdbf4eb375b4f1d9
commit r15-3756-gb6ea4f7b0e6f983376aac6ebcdbf4eb375b4f1d9
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #26 from GCC Commits ---
The master branch has been updated by Mikael Morin :
https://gcc.gnu.org/g:053cec9a5f4513537ca20a246c39d1bbfe6ce4be
commit r15-3757-g053cec9a5f4513537ca20a246c39d1bbfe6ce4be
Author: Mikael Morin
Date: Sat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #24 from Tamar Christina ---
(In reply to Mikael Morin from comment #23)
> (In reply to Mikael Morin from comment #21)
> >
> > (...) and should be able to submit the first
> > series (inline minloc without dim argument) this week.
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #23 from Mikael Morin ---
(In reply to Mikael Morin from comment #21)
>
> (...) and should be able to submit the first
> series (inline minloc without dim argument) this week.
>
I missed the "this week" mark (again), but I've finall
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #22 from Tamar Christina ---
(In reply to Mikael Morin from comment #21)
> (In reply to Tamar Christina from comment #20)
> > Hi Mikael,
> >
> > I did regression testing on x86_64 and AArch64 and only found one test-ism.
> >
> > I t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #21 from Mikael Morin ---
(In reply to Tamar Christina from comment #20)
> Hi Mikael,
>
> I did regression testing on x86_64 and AArch64 and only found one test-ism.
>
> I think I understand most of the patch to be able to deal with
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #20 from Tamar Christina ---
Hi Mikael,
I did regression testing on x86_64 and AArch64 and only found one test-ism.
I think I understand most of the patch to be able to deal with any fallout,
would it be ok if I fix the test-ism and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #19 from Tamar Christina ---
Hi Mikael,
It looks like the last version of your patch already gets inlined in the call
sites we cared about.
Would it be possible for you to upstream it?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #18 from Tamar Christina ---
(In reply to Mikael Morin from comment #16)
> This missed the gcc stage 1 deadline, but I'm still working on it.
Thanks Mikael! If I can help with anything do let me know :)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #17 from anlauf at gcc dot gnu.org ---
(In reply to Mikael Morin from comment #16)
> This missed the gcc stage 1 deadline, but I'm still working on it.
I always thought that the Fortran FE does not fall under this rule.
Why don't you
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Mikael Morin changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #15 from Tamar Christina ---
(In reply to Mikael Morin from comment #14)
> Created attachment 56313 [details]
> inline minloc with mask
>
> This patch adds support for {min,max}loc with mask.
Awesome, thank you!
> It is not 100% te
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Mikael Morin changed:
What|Removed |Added
Attachment #56094|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #13 from Mikael Morin ---
(In reply to Tamar Christina from comment #12)
> (In reply to Mikael Morin from comment #11)
> > Created attachment 56094 [details]
> > Improved patch
> >
> > This improved patch (still single argument only)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #12 from Tamar Christina ---
(In reply to Mikael Morin from comment #11)
> Created attachment 56094 [details]
> Improved patch
>
> This improved patch (still single argument only) passes the fortran
> regression testsuite.
>
Awesom
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Mikael Morin changed:
What|Removed |Added
Attachment #56091|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #10 from Mikael Morin ---
(In reply to Mikael Morin from comment #8)
> (...) that is it was using too loops in a row in some cases.
>
... *two* loops in a row ...
(In reply to Tamar Christina from comment #9)
>
> Thanks Mikael!
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #9 from Tamar Christina ---
(In reply to Mikael Morin from comment #8)
> Created attachment 56091 [details]
> Rough patch
>
> Here is a rough patch to make the scalarizer support minloc calls.
> It regresses on minloc_1.f90 at least,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #8 from Mikael Morin ---
Created attachment 56091
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=56091&action=edit
Rough patch
Here is a rough patch to make the scalarizer support minloc calls.
It regresses on minloc_1.f90 at l
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Thomas Koenig changed:
What|Removed |Added
CC||mikael at gcc dot gnu.org
--- Comment #7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Tamar Christina changed:
What|Removed |Added
CC||tnfchris at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #5 from Thomas Koenig ---
(In reply to ktkachov from comment #4)
> LTO'ing libgfortran aside, how much work would it be to teach the scalarizer
> to at least elide the temporary arrays in expressions like:
> A(:) = minloc(...) ?
> I t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #4 from ktkachov at gcc dot gnu.org ---
LTO'ing libgfortran aside, how much work would it be to teach the scalarizer to
at least elide the temporary arrays in expressions like:
A(:) = minloc(...) ?
I think that would be a good start to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
--- Comment #3 from ktkachov at gcc dot gnu.org ---
(In reply to Thomas Koenig from comment #1)
> Another, not mutually exclusive approach would be to make libgfortran LTO
> clean so the more complex minloc etc calls could be pulled in.
Interesti
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90608
Thomas Koenig changed:
What|Removed |Added
See Also||https://gcc.gnu.org/bugzill
37 matches
Mail list logo
