This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH rs6000] Fix for builtins-4-runnable.c testcase FAIL on power7/BE 32-bit
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Carl Love <cel at us dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Mon, 12 Feb 2018 09:17:52 -0600
- Subject: Re: [PATCH rs6000] Fix for builtins-4-runnable.c testcase FAIL on power7/BE 32-bit
- Authentication-results: sourceware.org; auth=none
- References: <1518193377.7508.2.camel@us.ibm.com> <1518214146.7508.7.camel@us.ibm.com>
Hi Carl,
On Fri, Feb 09, 2018 at 02:09:06PM -0800, Carl Love wrote:
> As pointed out, the dg arguments in new test file was missing the
> {target 128}. I updated the arguments to be
>
> { dg-do run { target { int128 && powerpc64*-*-* } } }
>
> Without the powerpc64*-*-* the test was still tried to compiled the
> test case in 32-bit mode on BE and failed.
If the dg-do target clause fails, you still get the default dg-do value,
which is "compile" for most testcases.
You want to use dg-require-effective-target int128 if the testcase
cannot compile without int128. Could you try that?
Btw, your patch is completely whitespace-damaged which makes it very hard
to read (and impossible to apply). Please fix your setup :-)
Segher