https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121185

--- Comment #21 from Jürgen Reuter <juergen.reuter at desy dot de> ---
(In reply to anlauf from comment #18)
> (In reply to Jürgen Reuter from comment #17)
> > (In reply to anlauf from comment #16)
> > > (In reply to Jürgen Reuter from comment #15)
> > > > Created attachment 61950 [details]
> > > > Close-to-minimal reproducer, 186 lines
> > > > 
> > > > The division of the array pmatrix%value by trace is not performed 
> > > > elemental
> > > > but just on the first array element.
> > > 
> > > Great reproducer!
> > > 
> > > Changing the line
> > > 
> > >     pmatrix%value = pmatrix%value / trace * pmatrix%degree
> > > 
> > > to
> > > 
> > >     pmatrix%value(:) = pmatrix%value / trace * pmatrix%degree
> > > 
> > > to suppress the reallocation on assignment fixes the testcase for me.
> > > 
> > > Can you try to revert Mikael's recent changes and see if it fixes the
> > > regression?
> > 
> > Which commit by Mikael is that? must have been after Monday 0:01 or so.
> > I am still checking the second example with the VEGAS/VAMP integration
> > whether this is a similar problem or something else.
> 
> The following commits come to mind:
> 
> r16-2247, r16-2248, r16-2249

Actually, sorry, I cloned on 14:30 on Monday, so my head is:
$ git show HEAD
commit 47a7dcd4c027bcc98760043c3416aa354f42b40e (HEAD -> master, origin/trunk,
origin/master, origin/HEAD)
Author: Mikael Morin <mik...@gcc.gnu.org>
Date:   Tue Jul 15 19:09:21 2025 +0200

which includes exactly the recent commit by Mikael Morin. Our CI result from
midnight only shows one failing unit test (the VAMP/VEGAS) one, so from this I
conclude: Mikael's commit broke the code of the "pol" example here and there is
another problem. Is there any commit in the last week that would actually
change numerics?

Reply via email to