This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Replying to an older patch ([PATCH] Fix failing test-case)
- From: Martin Liška <mliska at suse dot cz>
- To: Sudakshina Das <sudi dot das at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: nd <nd at arm dot com>, hubika at ucw dot cz
- Date: Tue, 19 Dec 2017 14:20:41 +0100
- Subject: Re: Replying to an older patch ([PATCH] Fix failing test-case)
- Authentication-results: sourceware.org; auth=none
- References: <a45b078e-0d23-aeec-17bb-e2fdf1dd2c92@arm.com> <9ae1eb49-a759-8c7e-f8fd-01e5d0f245e0@suse.cz> <b8f007ee-c055-0186-e028-90037f1af81c@arm.com>
On 12/19/2017 02:03 PM, Sudakshina Das wrote:
> Hi Martin
>
> On 19/12/17 10:49, Martin Liška wrote:
>> On 11/30/2017 12:03 PM, Sudakshina Das wrote:
>>> https://gcc.gnu.org/ml/gcc-patches/2017-10/msg01157.html
>>>
>>> This patch fixed a test case switch-case-2.c. I am seeing switch-case-1.c failing on
>>>
>>> arm-none-linux-gnueabihf:
>>> FAIL: gcc.dg/tree-prof/switch-case-1.c scan-rtl-dump-times expand ";; basic block[^\\n]*count 2000" 1 (found 0 times)
>>>
>>> aarch64-none-linux-gnu:
>>> FAIL: gcc.dg/tree-prof/switch-case-1.c scan-rtl-dump-times expand ";; basic block[^\\n]*count 2000" 1 (found 2 times)
>>>
>>> which looks pretty similar (also the same changes make it pass). Can you confirm?
>>>
>>> Sudi
>>
>> Hello.
>>
>> There's patch for that. Can you please test it?
>>
>
> I have tested these changes and the test case passes for both arm-none-linux-gnueabihf and aarch64-none-linux-gnu.
Good, thanks for testing. It's quite obvious patch and I'm going to install it.
Martin
>
> Thanks
> Sudi
>
>> Martin
>>
>