On Fri, Jul 7, 2023 at 3:34 PM Jan Beulich <jbeul...@suse.com> wrote: > > On 07.07.2023 09:30, Hongtao Liu wrote: > > On Fri, Jul 7, 2023 at 3:13 PM Jan Beulich via Gcc-regression > > <gcc-regress...@gcc.gnu.org> wrote: > >> > >> On 06.07.2023 13:57, haochen.jiang wrote: > >>> On Linux/x86_64, > >>> > >>> 2d11c99dfca3cc603dbbfafb3afc41689a68e40f is the first bad commit > >>> commit 2d11c99dfca3cc603dbbfafb3afc41689a68e40f > >>> Author: Jan Beulich <jbeul...@suse.com> > >>> Date: Wed Jul 5 09:41:09 2023 +0200 > >>> > >>> x86: use VPTERNLOG also for certain andnot forms > >>> > >>> caused > >>> > >>> FAIL: gcc.target/i386/pr53652-1.c scan-assembler-not vpternlogq[ \\t] > >> > >> The respective expectation was never valid to add without excluding > >> cases where -march= overrides (extends) the -msse2 that the test > >> specifies explicitly. I'm afraid I don't know how to tweak a testcase > >> to properly deal with that. Perhaps (like iirc was suggested elsewhere) > >> -mno-avx512f, but honestly this approach feels clumsy to me. Cc-ing > >> Hongtao, who I think suggested that approach elsewhere. > >> > >>> FAIL: gcc.target/i386/pr53652-1.c scan-assembler-times pandn[ \\t] 2 > > There're a false dependence when using pternlog for andnot(and other > > newly added) pattern, i'm working on a patch to avoid that(PR110438). > > Let me handle the test case. > > Of course I'm happy to see you handle the testcase, but if you don't > mind I'm curious towards the connection you see between that false > dependency issue and the adjustments missing in this (and other) > testcase(s). For the sake of simplicity, add -mno-avx512f should be ok, the testcase is used to detect optimization on non-avx512 targets. I'll add extra testcases to cover false dependence case. > > Jan
-- BR, Hongtao
