On 09.04.2018 12:16, Andrey Belevantsev wrote:
> On 06.04.2018 18:59, Alexander Monakov wrote:
>> On Tue, 3 Apr 2018, Andrey Belevantsev wrote:
>>
>>> Hello,
>>>
>>> This issues is about the correct order in which we need to call the
>>> routines that fix up the control flow for us.
>>
>> OK with formatting both in the new comment and the Changelog fixed.
>
> Thanks, fixed that in rev. 259229.
I've found out that this patch also fixes PR 83852, so I've committed the
test from that PR as obvious after verifying that it works on cross-ppc
compiler and on x86-64.
Andrey
Index: gcc.dg/pr83852.c
===================================================================
*** gcc.dg/pr83852.c (revision 0)
--- gcc.dg/pr83852.c (revision 259373)
***************
*** 0 ****
--- 1,33 ----
+ /* { dg-do compile { target powerpc*-*-* ia64-*-* i?86-*-* x86_64-*-* } } */
+ /* { dg-options "-std=gnu99 -O2 -fselective-scheduling -fno-if-conversion
-fno-tree-dse -w" } */
+ long long int uo;
+ unsigned int vt;
+
+ void
+ r5 (long long int h8, long long int pu)
+ {
+ short int wj;
+ long long int *mh = h8;
+
+ for (wj = 0; wj < 3; ++wj)
+ {
+ int oq;
+ long long int ns, xf;
+
+ h8 += 2;
+ oq = !!h8 && !!wj;
+ ++uo;
+ vt ^= oq + uo;
+ ns = !!uo && !!vt;
+ xf = (h8 != 0) ? mh : 1;
+ pu += ns < xf;
+ }
+
+ for (pu = 0; pu < 1; ++pu)
+ {
+ int *sc;
+
+ sc = (int *)&pu;
+ *sc = 0;
+ }
+ }
Index: ChangeLog
===================================================================
*** ChangeLog (revision 259372)
--- ChangeLog (revision 259373)
***************
*** 1,3 ****
--- 1,8 ----
+ 2018-04-13 Andrey Belevantsev <a...@ispras.ru>
+
+ PR rtl-optimization/83852
+ * gcc.dg/pr83852.c: New testcase.
+
2018-04-13 Andreas Krebbel <kreb...@linux.ibm.com>
PR testsuite/85326
