This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix spec-options.c test case
- From: Bernd Edlinger <bernd dot edlinger at hotmail dot de>
- To: Oleg Endo <oleg dot endo at t-online dot de>, Bernd Schmidt <bschmidt at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>, Dominik Vogt <vogt at linux dot vnet dot ibm dot com>, Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- Date: Tue, 3 May 2016 00:05:06 +0000
- Subject: Re: [PATCH] Fix spec-options.c test case
- Authentication-results: sourceware.org; auth=none
- Authentication-results: spf=softfail (sender IP is 25.152.2.53) smtp.mailfrom=hotmail.de; t-online.de; dkim=none (message not signed) header.d=none;t-online.de; dmarc=none action=none header.from=hotmail.de;
- References: <AM4PR07MB1571BF44DAAEA78762C2F494E4780 at AM4PR07MB1571 dot eurprd07 dot prod dot outlook dot com> <57272B41 dot 8080703 at redhat dot com> <AM4PR07MB1571F18E8343708341E6D725E4790 at AM4PR07MB1571 dot eurprd07 dot prod dot outlook dot com> <57276071 dot 1010206 at redhat dot com> <1462224483 dot 31604 dot 120 dot camel at t-online dot de>
On 02.05.2016 23:28, Oleg Endo wrote:
> On Mon, 2016-05-02 at 16:13 +0200, Bernd Schmidt wrote:
>> On 05/02/2016 03:43 PM, Bernd Edlinger wrote:
>>> Yes, you are right. Only the original use-case seems to be
>>> sh-superh-elf specific. But there are also spec strings
>>> that are always available. I think adding -DFOO to
>>> "cpp_unique_options" will work on any target, and make the
>>> test case even more useful.
>>>
>>>
>>> So is the updated patch OK?
>>
>> If that passes testing on non-sh, yes.
>
> It seems that test case doesn't work on sh4-linux. There's a new
> failure:
>
> https://gcc.gnu.org/ml/gcc-testresults/2016-05/msg00089.html
>
> FAIL: gcc.dg/spec-options.c execution test
>
No, I actually fixed it, thanks. That test was done without my patch:
> LAST_UPDATED: Sun May 1 13:46:11 UTC 2016 (revision 235692)
svn log -r235762
------------------------------------------------------------------------
r235762 | edlinger | 2016-05-02 16:41:25 +0200 (Mo, 02. Mai 2016) | 5 Zeilen
2016-05-02 Bernd Edlinger <bernd.edlinger@hotmail.de>
* gcc.dg/spec-options.c: Run the test on all targets.
* gcc.dg/foo.specs: Use cpp_unique_options.
You see: 235692 < 235762
As I said the test case _was_ bogus from the beginning, and would
_only_ pass for sh*-superh-elf target, but it was never passing for
--target=sh4-unknown-linux-gnu. However we were not aware of that
fact, because the dejagnu bug masked that until the two contradicting
dg-do rules were swapped.
I'd bet that the rest of the handful of testcases where two different
dg-do statements are used, are also bogus, because they have never been
executed due to the dejagu-bug, and they are written differently than
all the other tests.
Bernd.