On Thu, Oct 13, 2011 at 4:55 AM, Richard Guenther <rguent...@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 <rguent...@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.
--
H.J.
---
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...@intel.com>
+
+ * gcc.target/i386/pr50712.c: Check ia32 instead of ilp32.
+
2011-10-13 Eric Botcazou <ebotca...@adacore.com>
* gcc.dg/builtins-67.c: Guard iround and irint with HAVE_C99_RUNTIME.
