* [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 @ 2023-07-06 11:57 haochen.jiang 2023-07-07 7:17 ` Jan Beulich 0 siblings, 1 reply; 7+ messages in thread From: haochen.jiang @ 2023-07-06 11:57 UTC (permalink / raw) To: jbeulich, gcc-regression, gcc-patches, haochen.jiang On Linux/x86_64, e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 Author: Jan Beulich <jbeulich@suse.com> Date: Wed Jul 5 09:49:16 2023 +0200 x86: yet more PR target/100711-like splitting caused FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 with GCC configured with ../../gcc/configure --prefix=/export/users/haochenj/src/gcc-bisect/master/master/r14-2314/usr --enable-clocale=gnu --with-system-zlib --with-demangler-in-ld --with-fpmath=sse --enable-languages=c,c++,fortran --enable-cet --without-isl --enable-libmpx x86_64-linux --disable-bootstrap To reproduce: $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-1.c --target_board='unix{-m32\ -march=cascadelake}'" $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-2.c --target_board='unix{-m32\ -march=cascadelake}'" (Please do not reply to this email, for question about this report, contact me at haochen dot jiang at intel.com) ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-06 11:57 [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 haochen.jiang @ 2023-07-07 7:17 ` Jan Beulich 2023-07-07 7:46 ` Hongtao Liu 0 siblings, 1 reply; 7+ messages in thread From: Jan Beulich @ 2023-07-07 7:17 UTC (permalink / raw) To: haochen.jiang, haochen.jiang; +Cc: gcc-regression, gcc-patches, Hongtao Liu On 06.07.2023 13:57, haochen.jiang wrote: > On Linux/x86_64, > > e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit > commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 > Author: Jan Beulich <jbeulich@suse.com> > Date: Wed Jul 5 09:49:16 2023 +0200 > > x86: yet more PR target/100711-like splitting > > caused > > FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 > FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 I expect the same applies here - -mno-avx512f (or -mno-avx512vl) might address this failure. But whether that's really the way to go I'm not sure of. Plus of course such adjustments should have been done ahead of time, when it was decided that testing with certain -march= settings is a goal. My changes have merely uncovered the prior omissions. Jan > with GCC configured with > > ../../gcc/configure --prefix=/export/users/haochenj/src/gcc-bisect/master/master/r14-2314/usr --enable-clocale=gnu --with-system-zlib --with-demangler-in-ld --with-fpmath=sse --enable-languages=c,c++,fortran --enable-cet --without-isl --enable-libmpx x86_64-linux --disable-bootstrap > > To reproduce: > > $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-1.c --target_board='unix{-m32\ -march=cascadelake}'" > $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-2.c --target_board='unix{-m32\ -march=cascadelake}'" > > (Please do not reply to this email, for question about this report, contact me at haochen dot jiang at intel.com) ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-07 7:17 ` Jan Beulich @ 2023-07-07 7:46 ` Hongtao Liu 2023-07-07 7:49 ` Jan Beulich 0 siblings, 1 reply; 7+ messages in thread From: Hongtao Liu @ 2023-07-07 7:46 UTC (permalink / raw) To: Jan Beulich Cc: haochen.jiang, haochen.jiang, gcc-regression, gcc-patches, Hongtao Liu On Fri, Jul 7, 2023 at 3:18 PM Jan Beulich via Gcc-regression <gcc-regression@gcc.gnu.org> wrote: > > On 06.07.2023 13:57, haochen.jiang wrote: > > On Linux/x86_64, > > > > e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit > > commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 > > Author: Jan Beulich <jbeulich@suse.com> > > Date: Wed Jul 5 09:49:16 2023 +0200 > > > > x86: yet more PR target/100711-like splitting > > > > caused > > > > FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 > > FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 > > I expect the same applies here - -mno-avx512f (or -mno-avx512vl) might For this one, we can just add -mno-avx512f to the testcase,it aims to optimize pandn for avx2 target. > address this failure. But whether that's really the way to go I'm not > sure of. Plus of course such adjustments should have been done ahead > of time, when it was decided that testing with certain -march= settings > is a goal. My changes have merely uncovered the prior omissions. It's not a standard request, it's just our private tester which is used to find gcc bugs and miss-optimizations. It sometimes generates false positive reports (usually adding -mno-avx512f to the testcase can fix that), hope that's not too annoying. > > Jan > > > with GCC configured with > > > > ../../gcc/configure --prefix=/export/users/haochenj/src/gcc-bisect/master/master/r14-2314/usr --enable-clocale=gnu --with-system-zlib --with-demangler-in-ld --with-fpmath=sse --enable-languages=c,c++,fortran --enable-cet --without-isl --enable-libmpx x86_64-linux --disable-bootstrap > > > > To reproduce: > > > > $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-1.c --target_board='unix{-m32\ -march=cascadelake}'" > > $ cd {build_dir}/gcc && make check RUNTESTFLAGS="i386.exp=gcc.target/i386/pr100711-2.c --target_board='unix{-m32\ -march=cascadelake}'" > > > > (Please do not reply to this email, for question about this report, contact me at haochen dot jiang at intel.com) > -- BR, Hongtao ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-07 7:46 ` Hongtao Liu @ 2023-07-07 7:49 ` Jan Beulich 2023-07-07 7:50 ` Hongtao Liu 0 siblings, 1 reply; 7+ messages in thread From: Jan Beulich @ 2023-07-07 7:49 UTC (permalink / raw) To: Hongtao Liu Cc: haochen.jiang, haochen.jiang, gcc-regression, gcc-patches, Hongtao Liu On 07.07.2023 09:46, Hongtao Liu wrote: > On Fri, Jul 7, 2023 at 3:18 PM Jan Beulich via Gcc-regression > <gcc-regression@gcc.gnu.org> wrote: >> >> On 06.07.2023 13:57, haochen.jiang wrote: >>> On Linux/x86_64, >>> >>> e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit >>> commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 >>> Author: Jan Beulich <jbeulich@suse.com> >>> Date: Wed Jul 5 09:49:16 2023 +0200 >>> >>> x86: yet more PR target/100711-like splitting >>> >>> caused >>> >>> FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 >>> FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 >> >> I expect the same applies here - -mno-avx512f (or -mno-avx512vl) might > For this one, we can just add -mno-avx512f to the testcase,it aims to > optimize pandn for avx2 target. >> address this failure. But whether that's really the way to go I'm not >> sure of. Plus of course such adjustments should have been done ahead >> of time, when it was decided that testing with certain -march= settings >> is a goal. My changes have merely uncovered the prior omissions. > It's not a standard request, it's just our private tester which is > used to find gcc bugs and miss-optimizations. > It sometimes generates false positive reports (usually adding > -mno-avx512f to the testcase can fix that), hope that's not too > annoying. Wouldn't that then better be done once uniformly for all affected tests, rather than being discovered piecemeal? Anyway, in this case: Since you said you'd take care of the other test, will/can you do so for the two ones here as well, or am I on the hook? Jan ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-07 7:49 ` Jan Beulich @ 2023-07-07 7:50 ` Hongtao Liu 2023-07-07 7:55 ` Hongtao Liu 0 siblings, 1 reply; 7+ messages in thread From: Hongtao Liu @ 2023-07-07 7:50 UTC (permalink / raw) To: Jan Beulich Cc: haochen.jiang, haochen.jiang, gcc-regression, gcc-patches, Hongtao Liu On Fri, Jul 7, 2023 at 3:50 PM Jan Beulich <jbeulich@suse.com> wrote: > > On 07.07.2023 09:46, Hongtao Liu wrote: > > On Fri, Jul 7, 2023 at 3:18 PM Jan Beulich via Gcc-regression > > <gcc-regression@gcc.gnu.org> wrote: > >> > >> On 06.07.2023 13:57, haochen.jiang wrote: > >>> On Linux/x86_64, > >>> > >>> e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit > >>> commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 > >>> Author: Jan Beulich <jbeulich@suse.com> > >>> Date: Wed Jul 5 09:49:16 2023 +0200 > >>> > >>> x86: yet more PR target/100711-like splitting > >>> > >>> caused > >>> > >>> FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 > >>> FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 > >> > >> I expect the same applies here - -mno-avx512f (or -mno-avx512vl) might > > For this one, we can just add -mno-avx512f to the testcase,it aims to > > optimize pandn for avx2 target. > >> address this failure. But whether that's really the way to go I'm not > >> sure of. Plus of course such adjustments should have been done ahead > >> of time, when it was decided that testing with certain -march= settings > >> is a goal. My changes have merely uncovered the prior omissions. > > It's not a standard request, it's just our private tester which is > > used to find gcc bugs and miss-optimizations. > > It sometimes generates false positive reports (usually adding > > -mno-avx512f to the testcase can fix that), hope that's not too > > annoying. > > Wouldn't that then better be done once uniformly for all affected tests, > rather than being discovered piecemeal? > > Anyway, in this case: Since you said you'd take care of the other test, > will/can you do so for the two ones here as well, or am I on the hook? I'll do that. > > Jan -- BR, Hongtao ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-07 7:50 ` Hongtao Liu @ 2023-07-07 7:55 ` Hongtao Liu 2023-07-07 8:08 ` Jiang, Haochen 0 siblings, 1 reply; 7+ messages in thread From: Hongtao Liu @ 2023-07-07 7:55 UTC (permalink / raw) To: Jan Beulich Cc: haochen.jiang, haochen.jiang, gcc-regression, gcc-patches, Hongtao Liu On Fri, Jul 7, 2023 at 3:50 PM Hongtao Liu <crazylht@gmail.com> wrote: > > On Fri, Jul 7, 2023 at 3:50 PM Jan Beulich <jbeulich@suse.com> wrote: > > > > On 07.07.2023 09:46, Hongtao Liu wrote: > > > On Fri, Jul 7, 2023 at 3:18 PM Jan Beulich via Gcc-regression > > > <gcc-regression@gcc.gnu.org> wrote: > > >> > > >> On 06.07.2023 13:57, haochen.jiang wrote: > > >>> On Linux/x86_64, > > >>> > > >>> e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit > > >>> commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 > > >>> Author: Jan Beulich <jbeulich@suse.com> > > >>> Date: Wed Jul 5 09:49:16 2023 +0200 > > >>> > > >>> x86: yet more PR target/100711-like splitting > > >>> > > >>> caused > > >>> > > >>> FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 > > >>> FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 > > >> > > >> I expect the same applies here - -mno-avx512f (or -mno-avx512vl) might > > > For this one, we can just add -mno-avx512f to the testcase,it aims to > > > optimize pandn for avx2 target. > > >> address this failure. But whether that's really the way to go I'm not > > >> sure of. Plus of course such adjustments should have been done ahead > > >> of time, when it was decided that testing with certain -march= settings > > >> is a goal. My changes have merely uncovered the prior omissions. > > > It's not a standard request, it's just our private tester which is > > > used to find gcc bugs and miss-optimizations. > > > It sometimes generates false positive reports (usually adding > > > -mno-avx512f to the testcase can fix that), hope that's not too > > > annoying. > > > > Wouldn't that then better be done once uniformly for all affected tests, > > rather than being discovered piecemeal? This also prevents us from finding potential problems. > > > > Anyway, in this case: Since you said you'd take care of the other test, > > will/can you do so for the two ones here as well, or am I on the hook? > I'll do that. > > > > Jan > > > > -- > BR, > Hongtao -- BR, Hongtao ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 2023-07-07 7:55 ` Hongtao Liu @ 2023-07-07 8:08 ` Jiang, Haochen 0 siblings, 0 replies; 7+ messages in thread From: Jiang, Haochen @ 2023-07-07 8:08 UTC (permalink / raw) To: Hongtao Liu, Beulich, Jan Cc: haochen.jiang, gcc-regression, gcc-patches, Liu, Hongtao > -----Original Message----- > From: Hongtao Liu <crazylht@gmail.com> > Sent: Friday, July 7, 2023 3:55 PM > To: Beulich, Jan <JBeulich@suse.com> > Cc: haochen.jiang <haochenj@ecsmtp.sh.intel.com>; Jiang, Haochen > <haochen.jiang@intel.com>; gcc-regression@gcc.gnu.org; gcc- > patches@gcc.gnu.org; Liu, Hongtao <hongtao.liu@intel.com> > Subject: Re: [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan- > assembler-times vpandn 8 on Linux/x86_64 > > On Fri, Jul 7, 2023 at 3:50 PM Hongtao Liu <crazylht@gmail.com> wrote: > > > > On Fri, Jul 7, 2023 at 3:50 PM Jan Beulich <jbeulich@suse.com> wrote: > > > > > > On 07.07.2023 09:46, Hongtao Liu wrote: > > > > On Fri, Jul 7, 2023 at 3:18 PM Jan Beulich via Gcc-regression > > > > <gcc-regression@gcc.gnu.org> wrote: > > > >> > > > >> On 06.07.2023 13:57, haochen.jiang wrote: > > > >>> On Linux/x86_64, > > > >>> > > > >>> e007369c8b67bcabd57c4fed8cff2a6db82e78e6 is the first bad commit > > > >>> commit e007369c8b67bcabd57c4fed8cff2a6db82e78e6 > > > >>> Author: Jan Beulich <jbeulich@suse.com> > > > >>> Date: Wed Jul 5 09:49:16 2023 +0200 > > > >>> > > > >>> x86: yet more PR target/100711-like splitting > > > >>> > > > >>> caused > > > >>> > > > >>> FAIL: gcc.target/i386/pr100711-1.c scan-assembler-times pandn 2 > > > >>> FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 > > > >> > > > >> I expect the same applies here - -mno-avx512f (or -mno-avx512vl) > > > >> might > > > > For this one, we can just add -mno-avx512f to the testcase,it aims > > > > to optimize pandn for avx2 target. > > > >> address this failure. But whether that's really the way to go I'm > > > >> not sure of. Plus of course such adjustments should have been > > > >> done ahead of time, when it was decided that testing with certain > > > >> -march= settings is a goal. My changes have merely uncovered the > prior omissions. > > > > It's not a standard request, it's just our private tester which is > > > > used to find gcc bugs and miss-optimizations. > > > > It sometimes generates false positive reports (usually adding > > > > -mno-avx512f to the testcase can fix that), hope that's not too > > > > annoying. > > > > > > Wouldn't that then better be done once uniformly for all affected > > > tests, rather than being discovered piecemeal? > This also prevents us from finding potential problems. Yes, -march=cascadelake actually opens AVX512F related features. It sometimes show the potential problems while sometimes false positive. I will add a hint in the script email. Thx, Haochen > > > > > > Anyway, in this case: Since you said you'd take care of the other > > > test, will/can you do so for the two ones here as well, or am I on the hook? > > I'll do that. > > > > > > Jan > > > > > > > > -- > > BR, > > Hongtao > > > > -- > BR, > Hongtao ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-07-07 8:08 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-07-06 11:57 [r14-2314 Regression] FAIL: gcc.target/i386/pr100711-2.c scan-assembler-times vpandn 8 on Linux/x86_64 haochen.jiang 2023-07-07 7:17 ` Jan Beulich 2023-07-07 7:46 ` Hongtao Liu 2023-07-07 7:49 ` Jan Beulich 2023-07-07 7:50 ` Hongtao Liu 2023-07-07 7:55 ` Hongtao Liu 2023-07-07 8:08 ` Jiang, Haochen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).