public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations [not found] <bug-40106-4@http.gcc.gnu.org/bugzilla/> @ 2010-10-01 12:01 ` jakub at gcc dot gnu.org 2011-04-16 10:41 ` jakub at gcc dot gnu.org 2012-03-13 16:15 ` jakub at gcc dot gnu.org 2 siblings, 0 replies; 6+ messages in thread From: jakub at gcc dot gnu.org @ 2010-10-01 12:01 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.4.5 |4.4.6 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations [not found] <bug-40106-4@http.gcc.gnu.org/bugzilla/> 2010-10-01 12:01 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations jakub at gcc dot gnu.org @ 2011-04-16 10:41 ` jakub at gcc dot gnu.org 2012-03-13 16:15 ` jakub at gcc dot gnu.org 2 siblings, 0 replies; 6+ messages in thread From: jakub at gcc dot gnu.org @ 2011-04-16 10:41 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.4.6 |4.4.7 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations [not found] <bug-40106-4@http.gcc.gnu.org/bugzilla/> 2010-10-01 12:01 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations jakub at gcc dot gnu.org 2011-04-16 10:41 ` jakub at gcc dot gnu.org @ 2012-03-13 16:15 ` jakub at gcc dot gnu.org 2 siblings, 0 replies; 6+ messages in thread From: jakub at gcc dot gnu.org @ 2012-03-13 16:15 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 Jakub Jelinek <jakub at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED Target Milestone|4.4.7 |4.5.0 --- Comment #60 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-03-13 13:04:15 UTC --- Fixed in 4.5+, 4.4 is no longer supported. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug middle-end/40106] New: Time increase with inlining for the Polyhedron test air.f90
@ 2009-05-11 18:04 dominiq at lps dot ens dot fr
2010-03-22 12:39 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations rguenth at gcc dot gnu dot org
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: dominiq at lps dot ens dot fr @ 2009-05-11 18:04 UTC (permalink / raw)
To: gcc-bugs
The run time of air.f90 of the Polyhedron test suite takes ~15% more time when
compiled with -fwhole-file than without the option. I have checked that the
subroutines DERIV(X|Y) are inlined with -finline-limit=100, but not with
-finline-limit=50 (for the later I recover the timing without -fwhole-file).
What I have found very odd is that if I manually inline only a single call (see
below) I get the same timing that with all of them (2*14) inlined. This is the
case for trunk and gfortran 4.4.0, but not for 4.3.3 which gives a slower
executable.
I have inlined
SUBROUTINE DERIVX(D,U,Ux,Al,Np,Nd,M)
IMPLICIT REAL*8(A-H,O-Z)
PARAMETER (NX=150,NY=150)
DIMENSION D(NX,33) , U(NX,NY) , Ux(NX,NY) , Al(30) , Np(30)
DO jm = 1 , M
jmax = 0
jmin = 1
DO i = 1 , Nd
jmax = jmax + Np(i) + 1
DO j = jmin , jmax
uxt = 0.
DO k = 0 , Np(i)
uxt = uxt + D(j,k+1)*U(jmin+k,jm)
ENDDO
Ux(j,jm) = uxt*Al(i)
ENDDO
!
jmin = jmin + Np(i) + 1
ENDDO
ENDDO
CONTINUE
END
at line 793 as
! CALL DERIVX(DX,f4,f4x,ALX,NPX,NDX,MXPy)
DO jm = 1 , MXPy
jmax = 0
jmin = 1
DO i = 1 , NDX
jmax = jmax + NPX(i) + 1
DO j = jmin , jmax
uxt = 0.
DO k = 0 , NPX(i)
uxt = uxt + DX(j,k+1)*f4(jmin+k,jm)
ENDDO
f4x(j,jm) = uxt*ALX(i)
ENDDO
jmin = jmin + NPX(i) + 1
ENDDO
ENDDO
--
Summary: Time increase with inlining for the Polyhedron test
air.f90
Product: gcc
Version: 4.5.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: middle-end
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: dominiq at lps dot ens dot fr
GCC build triplet: i686-apple-darwin9
GCC host triplet: i686-apple-darwin9
GCC target triplet: i686-apple-darwin9
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations 2009-05-11 18:04 [Bug middle-end/40106] New: Time increase with inlining for the Polyhedron test air.f90 dominiq at lps dot ens dot fr @ 2010-03-22 12:39 ` rguenth at gcc dot gnu dot org 2010-03-25 17:38 ` hubicka at gcc dot gnu dot org 2010-04-30 9:01 ` jakub at gcc dot gnu dot org 2 siblings, 0 replies; 6+ messages in thread From: rguenth at gcc dot gnu dot org @ 2010-03-22 12:39 UTC (permalink / raw) To: gcc-bugs ------- Comment #58 from rguenth at gcc dot gnu dot org 2010-03-22 12:39 ------- Fixed for 4.5. -- rguenth at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|rguenth at gcc dot gnu dot |unassigned at gcc dot gnu |org |dot org Status|ASSIGNED |NEW Known to work| |4.5.0 Summary|[4.4/4.5 Regression] Weird |[4.4 Regression] Weird |interaction between |interaction between |optimize_insn_for_speed_p |optimize_insn_for_speed_p |and -funsafe-math- |and -funsafe-math- |optimizations |optimizations http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations 2009-05-11 18:04 [Bug middle-end/40106] New: Time increase with inlining for the Polyhedron test air.f90 dominiq at lps dot ens dot fr 2010-03-22 12:39 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations rguenth at gcc dot gnu dot org @ 2010-03-25 17:38 ` hubicka at gcc dot gnu dot org 2010-04-30 9:01 ` jakub at gcc dot gnu dot org 2 siblings, 0 replies; 6+ messages in thread From: hubicka at gcc dot gnu dot org @ 2010-03-25 17:38 UTC (permalink / raw) To: gcc-bugs ------- Comment #59 from hubicka at gcc dot gnu dot org 2010-03-25 17:37 ------- Hi, concerning the optimize_*_for_size and maybe_hot_*_p predicates, the idea is that maybe_hot/probably_cold care about the profile alone. So when optimizing for size, parts of program still can be considered hot and this can be used by optimizers if doing so does not increase code size (i.e. one can trade copy in hot block for copy in cold block even at -Os). optimize_*_for_size should be aware of the defaults - with -Os everything is by default optimized for size unless user asks otherwise and with ohter levels only probably cold sutuff (that is negation of maybe_hot) is optimized for size. Let me check if there are some problems, but I guess this is just problem with too many nested loops leading to too large frequency differences. Honza -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations 2009-05-11 18:04 [Bug middle-end/40106] New: Time increase with inlining for the Polyhedron test air.f90 dominiq at lps dot ens dot fr 2010-03-22 12:39 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations rguenth at gcc dot gnu dot org 2010-03-25 17:38 ` hubicka at gcc dot gnu dot org @ 2010-04-30 9:01 ` jakub at gcc dot gnu dot org 2 siblings, 0 replies; 6+ messages in thread From: jakub at gcc dot gnu dot org @ 2010-04-30 9:01 UTC (permalink / raw) To: gcc-bugs -- jakub at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.4.4 |4.4.5 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40106 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-03-13 16:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-40106-4@http.gcc.gnu.org/bugzilla/>
2010-10-01 12:01 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations jakub at gcc dot gnu.org
2011-04-16 10:41 ` jakub at gcc dot gnu.org
2012-03-13 16:15 ` jakub at gcc dot gnu.org
2009-05-11 18:04 [Bug middle-end/40106] New: Time increase with inlining for the Polyhedron test air.f90 dominiq at lps dot ens dot fr
2010-03-22 12:39 ` [Bug middle-end/40106] [4.4 Regression] Weird interaction between optimize_insn_for_speed_p and -funsafe-math-optimizations rguenth at gcc dot gnu dot org
2010-03-25 17:38 ` hubicka at gcc dot gnu dot org
2010-04-30 9:01 ` jakub at gcc dot gnu dot org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).