public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug rtl-optimization/67864] New: [6 Regression] CSiBE size regression @ 2015-10-06 10:30 rguenth at gcc dot gnu.org 2015-10-06 10:30 ` [Bug rtl-optimization/67864] " rguenth at gcc dot gnu.org ` (2 more replies) 0 siblings, 3 replies; 4+ messages in thread From: rguenth at gcc dot gnu.org @ 2015-10-06 10:30 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67864 Bug ID: 67864 Summary: [6 Regression] CSiBE size regression Product: gcc Version: 6.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: rtl-optimization Assignee: unassigned at gcc dot gnu.org Reporter: rguenth at gcc dot gnu.org CC: segher at kernel dot crashing.org Target Milestone: --- http://gcc.opensuse.org/CSiBE/ shows a big size regression between r228310 (good) and r228359 (bad). Flags used are -Os -fgnu89-inline, suspect is +2015-10-01 Segher Boessenkool <segher@kernel.crashing.org> + + * bb-reorder.c (reorder_basic_blocks): Use the algorithm selected + with flag_reorder_blocks_algorithm. + * common.opt (freorder-blocks-algorithm=): New flag. + (reorder_blocks_algorithm): New enum. + * flag-types.h (reorder_blocks_algorithm): New enum. + * opts.c (default_options_table): Use -freorder-blocks at -O1 and up, + and -freorder-blocks-algorithm=stc at -O2 and up (not at -Os). The size regression is from 3562398 bytes to 3617915 bytes. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/67864] [6 Regression] CSiBE size regression 2015-10-06 10:30 [Bug rtl-optimization/67864] New: [6 Regression] CSiBE size regression rguenth at gcc dot gnu.org @ 2015-10-06 10:30 ` rguenth at gcc dot gnu.org 2015-10-11 9:03 ` segher at gcc dot gnu.org 2015-10-12 8:14 ` rguenth at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: rguenth at gcc dot gnu.org @ 2015-10-06 10:30 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67864 Richard Biener <rguenth at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |6.0 ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/67864] [6 Regression] CSiBE size regression 2015-10-06 10:30 [Bug rtl-optimization/67864] New: [6 Regression] CSiBE size regression rguenth at gcc dot gnu.org 2015-10-06 10:30 ` [Bug rtl-optimization/67864] " rguenth at gcc dot gnu.org @ 2015-10-11 9:03 ` segher at gcc dot gnu.org 2015-10-12 8:14 ` rguenth at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: segher at gcc dot gnu.org @ 2015-10-11 9:03 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67864 --- Comment #1 from Segher Boessenkool <segher at gcc dot gnu.org> --- Author: segher Date: Sun Oct 11 09:03:09 2015 New Revision: 228692 URL: https://gcc.gnu.org/viewcvs?rev=228692&root=gcc&view=rev Log: bb-reorder: Improve the simple algorithm for -Os (PR67864) As the PR points out, the "simple" reorder algorithm makes bigger code than the STC algorithm did, for -Os, for x86. I now tested it for many different targets and it turns out to be worse everywhere. This simple patch tunes "simple" a bit; this makes it better than STC almost everywhere. The only exceptions (for the targets where I have results) are x86 and mn10300. For those targets it may be best to switch the default algorithm for -Os to STC. 2015-10-11 Segher Boessenkool <segher@kernel.crashing.org> PR rtl-optimization/67864 * gcc/bb-reorder (reorder_basic_blocks_simple): Prefer existing fallthrough edges for conditional jumps. Don't sort candidate edges if not optimizing for speed. Modified: trunk/gcc/ChangeLog trunk/gcc/bb-reorder.c ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug rtl-optimization/67864] [6 Regression] CSiBE size regression 2015-10-06 10:30 [Bug rtl-optimization/67864] New: [6 Regression] CSiBE size regression rguenth at gcc dot gnu.org 2015-10-06 10:30 ` [Bug rtl-optimization/67864] " rguenth at gcc dot gnu.org 2015-10-11 9:03 ` segher at gcc dot gnu.org @ 2015-10-12 8:14 ` rguenth at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: rguenth at gcc dot gnu.org @ 2015-10-12 8:14 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67864 --- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> --- No difference for x86 (but that may have been expected) ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-10-12 8:14 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-10-06 10:30 [Bug rtl-optimization/67864] New: [6 Regression] CSiBE size regression rguenth at gcc dot gnu.org 2015-10-06 10:30 ` [Bug rtl-optimization/67864] " rguenth at gcc dot gnu.org 2015-10-11 9:03 ` segher at gcc dot gnu.org 2015-10-12 8:14 ` rguenth at gcc dot gnu.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).