This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR50712
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Richard Guenther <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 13 Oct 2011 08:40:58 -0700
- Subject: Re: [PATCH] Fix PR50712
- References: <alpine.LNX.2.00.1110131353510.308@zhemvz.fhfr.qr>
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.
--
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.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.