Polyhedron performance regression
Paul Richard Thomas
paul.richard.thomas@gmail.com
Mon Nov 13 18:23:00 GMT 2006
FX,
>
>
> I'm ashamed that revision 118492
> (http://gcc.gnu.org/viewcvs?view=rev&revision=118492), which
> introduced this performance regression, is the MOD/MODULO patch by
> Paul Thomas and myself.
I realised from looking at the code for ac.f90, whilst on the plane last
night, that this must be the case. ac.f90 hinges on the random number
generator, which uses MOD.
I don't think that we should be ashamed about it. 32bit systems are
fine; I checked the Polyhedron suite before submitting and was
disappointed that it seemed to have little or no effect, other than
correcting the original PR!
Evidently there is something not very clever with the 64bit library.
Can Uros advise, please? In fact, is there a -ffast-math version
available or is it resorting to the ordinary library version? Detecting
the presence or not of the fast version, was what stopped me from
implementing this patch for so long.
>
> Still one thing I don't understand is the difference between Paul
> results and mine for the fwprop: it looks like it impacted with -O3
> -ftree-vectorize -fno-fast-math.
No, I did not understand it either. However, I was in a burning hurry
and had to resort to using bash > time ./a.out to do the comparison.
Maybe the figures are too dependent on whatever background stuff was
happening.
We have to make a decision quickly as to whether to wait for the library
to be put right, if it can be, or to comment out the platch for the time
being. Please note that I am unable to do a thing to trunk or 4.2,
since the firewall of the site that I am at will not have anything to do
with ssh.
Regards
Paul
More information about the Fortran
mailing list