diff mbox

Fix PR50712

Message ID CAMe9rOpO63gv3HFa+KV67Ad6Y+zjN4souG-a0LR76p9fo9qfWg@mail.gmail.com
State New
Headers show

Commit Message

H.J. Lu Oct. 13, 2011, 3:40 p.m. UTC 
On Thu, Oct 13, 2011 at 4:55 AM, Richard Guenther <rguenther@suse.de> wrote:
>
> This fixes PR50712, an issue with IPA split uncovered by adding
> verifier calls after it ... we need to also gimplify reads of
> register typed memory when passing it as argument.
>
> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
>
> Richard.
>
> 2011-10-13  Richard Guenther  <rguenther@suse.de>
>
>        PR tree-optimization/50712
>        * ipa-split.c (split_function): Always re-gimplify parameters
>        when they are not gimple vals before passing them.  Properly
>        check for type compatibility.
>
>        * gcc.target/i386/pr50712.c: New testcase.
>

This test is valid only for ia32, not ilp32. I checked in this patch
to fix it.
diff mbox

Patch

Index: gcc.target/i386/pr50712.c
===================================================================
--- gcc.target/i386/pr50712.c	(revision 179925)
+++ gcc.target/i386/pr50712.c	(working copy)
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-require-effective-target ilp32 } */
+/* { dg-require-effective-target ia32 } */
 /* { dg-options "-O2" } */

 typedef __builtin_va_list __va_list;
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 179925)
+++ ChangeLog	(working copy)
@@ -1,3 +1,7 @@ 
+2011-10-13  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* gcc.target/i386/pr50712.c: Check ia32 instead of ilp32.
+
 2011-10-13  Eric Botcazou  <ebotcazou@adacore.com>

 	* gcc.dg/builtins-67.c: Guard iround and irint with HAVE_C99_RUNTIME.