public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Allow inlining across -fstrict-aliasing boundary
@ 2015-04-30 15:17 Jan Hubicka
  0 siblings, 0 replies; only message in thread
From: Jan Hubicka @ 2015-04-30 15:17 UTC (permalink / raw)
  To: glandium, gcc-patches

Hi,
this patch permits inlining across flag_strict_aliasing this is hopefully
safe because te memory accesses in -fstrict-aliasing should have alias
set 0.

This should alllow to build packages that currently fail on always_inline
because of use of explicit optimization attributes and hopefully also solve the
performance issues with Firefox and LTO+FDO build that crept in shortly before
release.

Bootstrapped/regtested x86_64-linux, will commit it to mainline and to branch
later next week if no problems shows up.

Honza

	PR ipa/65873
	* ipa-inline.c (can_inline_edge_p): It is safe to inline across
	-fstrict-aliasing boundaries.
Index: ipa-inline.c
===================================================================
--- ipa-inline.c	(revision 222620)
+++ ipa-inline.c	(working copy)
@@ -439,9 +439,6 @@ can_inline_edge_p (struct cgraph_edge *e
 	       == !opt_for_fn (callee->decl, optimize) || !always_inline))
 	  || check_match (flag_wrapv)
 	  || check_match (flag_trapv)
-	  /* Strictly speaking only when the callee contains memory
-	     accesses that are not using alias-set zero anyway.  */
-	  || check_maybe_down (flag_strict_aliasing)
 	  /* Strictly speaking only when the callee uses FP math.  */
 	  || check_maybe_up (flag_rounding_math)
 	  || check_maybe_up (flag_trapping_math)

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2015-04-30 15:11 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-30 15:17 Allow inlining across -fstrict-aliasing boundary Jan Hubicka

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).