This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Compile g++.dg/opt/pr52727.C with -march=i686 for ia32
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 12 Feb 2014 08:00:46 -0800
- Subject: Re: PATCH: Compile g++.dg/opt/pr52727.C with -march=i686 for ia32
- Authentication-results: sourceware.org; auth=none
- References: <20140204164405 dot GA27145 at intel dot com>
On Tue, Feb 4, 2014 at 8:44 AM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> Hi,
>
> Revision 206943 caused a regression on g++.dg/opt/pr52727.C, which wasn't
> caught by my automated tester since SSE2 and SSE math are enabled by
> default. This patch adds -march=i686 for ia32 when compiling
> g++.dg/opt/pr52727.C, which is required to reproduce the bug. OK to
> install?
>
> Thanks.
>
>
> H.J.
> ---
> 2014-02-04 H.J. Lu <hongjiu.lu@intel.com>
>
> * g++.dg/opt/pr52727.C: Compile with -march=i686 for ia32.
>
> diff --git a/gcc/testsuite/g++.dg/opt/pr52727.C b/gcc/testsuite/g++.dg/opt/pr52727.C
> index 4dd3853..ed8b973 100644
> --- a/gcc/testsuite/g++.dg/opt/pr52727.C
> +++ b/gcc/testsuite/g++.dg/opt/pr52727.C
> @@ -1,5 +1,6 @@
> // { dg-do compile }
> // { dg-options "-g -Os" }
> +// { dg-additional-options "-march=i686" { target ia32 } }
>
> int grow (int);
> void fn (int);
Since -march=i686 is required for this testcase on x86, I
am checking in this as obvious fix.
--
H.J.