From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20080 invoked by alias); 26 Apr 2013 10:00:13 -0000 Mailing-List: contact glibc-bugs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: glibc-bugs-owner@sourceware.org Received: (qmail 20003 invoked by uid 48); 26 Apr 2013 10:00:10 -0000 From: "siddhesh at redhat dot com" To: glibc-bugs@sourceware.org Subject: [Bug math/14412] Removal of sysdeps/x86_64/fpu/s_sincos.S causes regressions Date: Fri, 26 Apr 2013 10:00:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: glibc X-Bugzilla-Component: math X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: siddhesh at redhat dot com X-Bugzilla-Status: NEW X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at sourceware dot org X-Bugzilla-Target-Milestone: 2.18 X-Bugzilla-Changed-Fields: Status Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://sourceware.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 X-SW-Source: 2013-04/txt/msg00170.txt.bz2 http://sourceware.org/bugzilla/show_bug.cgi?id=14412 Siddhesh Poyarekar changed: What |Removed |Added ---------------------------------------------------------------------------- Status|WAITING |NEW --- Comment #18 from Siddhesh Poyarekar 2013-04-26 10:00:09 UTC --- (In reply to comment #16) > (In reply to comment #13) > > This should have been fixed with the patch for bug 14496. Markus, can you > > confirm that? > > It now takes ~40 seconds to render the test-page on trunk. > Better than 60 seconds but still 40% slower than s_sincos.S. That's expected. The assembly implementation used the fsincos instruction and the current implementation calls the sin() and cos() functions. The rounding mode changes have nothing to do with this part of the performance regression. Maybe we could implement a sincos_finite that calls the fsincos instruction and actually gets used with -ffinite-math. (In reply to comment #17) > Perhaps we should add a predefined macro for -frounding-math in gcc, and the > precise variant of libm could be selected using that macro, if the correct > rounding precise variants would be suffixed differently (like the *_finite > entrypoints are for -ffast-math), then all the 3 libm variants could live in > the same libm.so.6 and people would just choose what they want using > -ffast-math vs. default vs. -frounding-math. I like this idea. -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.