On 09.04.2018 21:23, Jakub Jelinek wrote:
> On Mon, Apr 09, 2018 at 01:30:12PM +0300, Andrey Belevantsev wrote:
>> I think that should be fine, that is, as long as the insn moved up through
>> all those debug insns, the copy will do that as well. It's that
>> problematic conditional in sched-deps.c that we should take care of.
>>
>> I've reworded the comment and committed the attached patch. Thanks for
>> your help.
>
> The C++ testcase FAILs everywhere:
> FAIL: g++.dg/pr80463.C -std=gnu++98 (test for excess errors)
> Excess errors:
> cc1plus: warning: var-tracking-assignments changes selective scheduling
>
> The other testcases in the patch used -w probably to disable the same
> warning, so I've committed following as obvious to trunk after regtesting it
> on x86_64-linux and i686-linux:
Thank you very much, I have missed this when committing. Sorry for the noise.
Andrey
>
> 2018-04-09 Jakub Jelinek <ja...@redhat.com>
>
> PR rtl-optimization/80463
> * g++.dg/pr80463.C: Add -w to dg-options.
>
> --- gcc/testsuite/g++.dg/pr80463.C.jj 2018-04-09 20:15:47.226631780 +0200
> +++ gcc/testsuite/g++.dg/pr80463.C 2018-04-09 20:19:43.783616136 +0200
> @@ -1,5 +1,5 @@
> /* { dg-do compile { target powerpc*-*-* ia64-*-* i?86-*-* x86_64-*-* } } */
> -/* { dg-options "-g -fselective-scheduling2 -O2 -fvar-tracking-assignments"
> } */
> +/* { dg-options "-g -fselective-scheduling2 -O2 -fvar-tracking-assignments
> -w" } */
>
> int *a;
> int b, c;
>
>
> Jakub
>
