On Fri, 23 Sep 2022, Jakub Jelinek wrote:
> On Thu, Sep 22, 2022 at 01:10:08PM +0200, Richard Biener via Gcc-patches
> wrote:
> > * g++.dg/tree-ssa/pr106922.C: Adjust.
>
> > --- a/gcc/testsuite/g++.dg/tree-ssa/pr106922.C
> > +++ b/gcc/testsuite/g++.dg/tree-ssa/pr106922.C
> > @@ -87,5 +87,4 @@ void testfunctionfoo() {
> > }
> > }
> >
> > -// { dg-final { scan-tree-dump-times "Found fully redundant value" 4 "pre"
> > { xfail { ! lp64 } } } }
> > -// { dg-final { scan-tree-dump-not "m_initialized" "cddce3" { xfail { !
> > lp64 } } } }
> > +// { dg-final { scan-tree-dump-not "m_initialized" "dce3" } }
>
> I've noticed
> +UNRESOLVED: g++.dg/tree-ssa/pr106922.C -std=gnu++20 scan-tree-dump-not
> dce3 "m_initialized"
> +UNRESOLVED: g++.dg/tree-ssa/pr106922.C -std=gnu++2b scan-tree-dump-not
> dce3 "m_initialized"
> with this change, both on x86_64 and i686.
> The dump is still cddce3, additionally as the last reference to the pre
> dump is gone, not sure it is worth creating that dump.
oops...
> With the following patch, there aren't FAILs nor UNRESOLVED tests with
> GXX_TESTSUITE_STDS=98,11,14,17,20,2b make check-g++
> RUNTESTFLAGS="--target_board=unix\{-m32,-m64\} dg.exp='pr106922.C'"
>
> Ok for trunk?
OK.
Thanks,
Richard.
> 2022-09-23 Jakub Jelinek <ja...@redhat.com>
>
> PR tree-optimization/106922
> * g++.dg/tree-ssa/pr106922.C: Scan in cddce3 dump rather than
> dce3. Remove -fdump-tree-pre-details from dg-options.
>
> --- gcc/testsuite/g++.dg/tree-ssa/pr106922.C.jj 2022-09-23
> 09:02:57.011311664 +0200
> +++ gcc/testsuite/g++.dg/tree-ssa/pr106922.C 2022-09-23 09:41:06.348797951
> +0200
> @@ -1,5 +1,5 @@
> // { dg-require-effective-target c++20 }
> -// { dg-options "-O2 -fdump-tree-pre-details -fdump-tree-cddce3" }
> +// { dg-options "-O2 -fdump-tree-cddce3" }
>
> template <typename> struct __new_allocator {
> void deallocate(int *, int) { operator delete(0); }
> @@ -87,4 +87,4 @@ void testfunctionfoo() {
> }
> }
>
> -// { dg-final { scan-tree-dump-not "m_initialized" "dce3" } }
> +// { dg-final { scan-tree-dump-not "m_initialized" "cddce3" } }
>
>
> Jakub
>
>
--
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH, Frankenstrasse 146, 90461 Nuernberg,
Germany; GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman;
HRB 36809 (AG Nuernberg)
