Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-05 Thread luoxhu
On 2019/11/6 02:20, Joseph Myers wrote: > On Tue, 5 Nov 2019, Kewen.Lin wrote: > >> Very good point! Since gcc doesn't pursue 100% testsuite pass rate, I >> noticed >> there are a few failures exposed/caused by some PRs all the time. Could we >> just leave the test case there without any pre wo

Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-05 Thread Joseph Myers
On Tue, 5 Nov 2019, Kewen.Lin wrote: > Very good point! Since gcc doesn't pursue 100% testsuite pass rate, I noticed > there are a few failures exposed/caused by some PRs all the time. Could we > just leave the test case there without any pre workaround till the PR get > fixed? Yes, leaving th

Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-04 Thread Kewen.Lin
on 2019/11/5 上午6:57, Joseph Myers wrote: > On Mon, 4 Nov 2019, luoxhu wrote: > >> -finline-functions is enabled by default for O2 since r276469, update the >> test cases with -fno-inline-functions. >> >> v2: disable inlining for the failed cases. Add two more failed cases >> not listed in BZ. Te

Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-04 Thread Joseph Myers
On Mon, 4 Nov 2019, luoxhu wrote: > -finline-functions is enabled by default for O2 since r276469, update the > test cases with -fno-inline-functions. > > v2: disable inlining for the failed cases. Add two more failed cases > not listed in BZ. Tested on P8LE, P8BE and P9LE. If inlining (or oth

Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-04 Thread Segher Boessenkool
Hi! On Mon, Nov 04, 2019 at 11:29:56AM +0800, luoxhu wrote: > -finline-functions is enabled by default for O2 since r276469, update the > test cases with -fno-inline-functions. Jeff already okay'ed it; just some changelog nits: > 2019-10-30 Xiong Hu Luo > > PR92090 PR other/9209

Re: [PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-04 Thread Jeff Law
On 11/3/19 8:29 PM, luoxhu wrote: > -finline-functions is enabled by default for O2 since r276469, update the > test cases with -fno-inline-functions. > > v2: disable inlining for the failed cases. Add two more failed cases > not listed in BZ. Tested on P8LE, P8BE and P9LE. > > > gcc/testsuite

[PATCH v2] PR92090: Fix testcase failures by r276469

2019-11-03 Thread luoxhu
-finline-functions is enabled by default for O2 since r276469, update the test cases with -fno-inline-functions. v2: disable inlining for the failed cases. Add two more failed cases not listed in BZ. Tested on P8LE, P8BE and P9LE. gcc/testsuite/ChangeLog: 2019-10-30 Xiong Hu Luo