[Bug testsuite/100272] New: some incomplete dg-commands

[Bug testsuite/100272] New: some incomplete dg-commands

[gscfq@t-online.de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

            Bug ID: 100272
           Summary: some incomplete dg-commands
           Product: gcc
           Version: 12.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: gscfq@t-online.de
  Target Milestone: ---

Missing a starting "{" after "//" :

./gcc/testsuite/g++.dg/ext/flexary13.C:      { 0, { } };   // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 1, { 2 } };   // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 2, { 3, 4 } }; // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 123, i };   // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 456, { i } }; // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 3, { i, j, k } }; // dg-warning
"initialization of a flexible array member" }
./gcc/testsuite/g++.dg/ext/flexary13.C:      { 1, { 2 } };   // dg-warning
"initialization of a flexible array member" }

./gcc/testsuite/g++.dg/diagnostic/ptrtomem1.C:22:requires (sizeof(T)==1) //
dg-message {\[with T = int \(X::\*\)\[5\]\]} }



Missing both starting "{" and ending "}" (if desired at all) :

./gcc/testsuite/g++.dg/template/spec26.C:1:// dg-do run

./gcc/testsuite/gcc.dg/pr20126.c:1:/* dg-do run */
./gcc/testsuite/gcc.dg/pr20126.c:2:/* dg-options "-O2" */

./gcc/testsuite/gcc.dg/tree-ssa/pr20739.c:3:/* dg-do compile */
./gcc/testsuite/gcc.dg/tree-ssa/pr20739.c:4:/* dg-options "-O" */

./gcc/testsuite/gcc.dg/tree-ssa/predcom-1.c:50:/* dg-final {
scan-tree-dump-times "looparound ref" 1 "pcom" } */

./libffi/testsuite/libffi.complex/complex_int.c:79:  /* dg-output "-2,8i 2,8i,
x 1234 1234, y 11110 11110" */



This dg-command seems to be truncated at "+" :

./gcc/testsuite/g++.dg/ipa/pr45572-2.C:2:// { dg-options
"-finline-small-functions -findirect-inlining -finline-function+
---
./gcc/testsuite/g++.dg/ipa/pr45572-2.C:2:// { dg-options
"-finline-small-functions -findirect-inlining -finline-functions" }

[Bug testsuite/100272] some incomplete dg-commands

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org
   Last reconfirmed|                            |2021-04-27
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
(In reply to G. Steinmetz from comment #0)
> Missing a starting "{" after "//" :
> 
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 0, { } };   // dg-warning
> "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 1, { 2 } };   // dg-warning
> "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 2, { 3, 4 } }; // dg-warning
> "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 123, i };   // dg-warning
> "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 456, { i } }; // dg-warning
> "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 3, { i, j, k } }; //
> dg-warning "initialization of a flexible array member" }
> ./gcc/testsuite/g++.dg/ext/flexary13.C:      { 1, { 2 } };   // dg-warning
> "initialization of a flexible array member" }

The warnings don't appear - maybe it was a way to "xfail" them w/o xfailing...

> ./gcc/testsuite/g++.dg/diagnostic/ptrtomem1.C:22:requires (sizeof(T)==1) //
> dg-message {\[with T = int \(X::\*\)\[5\]\]} }

Fixing.

> 
> 
> Missing both starting "{" and ending "}" (if desired at all) :
> 
> ./gcc/testsuite/g++.dg/template/spec26.C:1:// dg-do run
> 
> ./gcc/testsuite/gcc.dg/pr20126.c:1:/* dg-do run */
> ./gcc/testsuite/gcc.dg/pr20126.c:2:/* dg-options "-O2" */
> 
> ./gcc/testsuite/gcc.dg/tree-ssa/pr20739.c:3:/* dg-do compile */
> ./gcc/testsuite/gcc.dg/tree-ssa/pr20739.c:4:/* dg-options "-O" */
> 
> ./gcc/testsuite/gcc.dg/tree-ssa/predcom-1.c:50:/* dg-final {
> scan-tree-dump-times "looparound ref" 1 "pcom" } */

xfail as well

> ./libffi/testsuite/libffi.complex/complex_int.c:79:  /* dg-output "-2,8i
> 2,8i, x 1234 1234, y 11110 11110" */
> 
> 
> 
> This dg-command seems to be truncated at "+" :
> 
> ./gcc/testsuite/g++.dg/ipa/pr45572-2.C:2:// { dg-options
> "-finline-small-functions -findirect-inlining -finline-function+
> ---
> ./gcc/testsuite/g++.dg/ipa/pr45572-2.C:2:// { dg-options
> "-finline-small-functions -findirect-inlining -finline-functions" }

[Bug testsuite/100272] some incomplete dg-commands

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:3bc0d418a5d214a8ba57857656ca5c618df1a4bb

commit r12-142-g3bc0d418a5d214a8ba57857656ca5c618df1a4bb
Author: Richard Biener <rguenther@suse.de>
Date:   Tue Apr 27 10:45:32 2021 +0200

    testsuite/100272 - fix some malformed dg directives

    The bug points out several malformed dg directives, the following
    fixes the obvious ones where the testcases keep working after the
    change.

    2021-04-27  Richard Biener  <rguenther@suse.de>

            PR testsuite/100272
            * g++.dg/diagnostic/ptrtomem1.C: Fix dg directives.
            * g++.dg/ipa/pr45572-2.C: Likewise.
            * g++.dg/template/spec26.C: Likewise.
            * gcc.dg/pr20126.c: Likewise.
            * gcc.dg/tree-ssa/pr20739.c: Likewise.

[Bug testsuite/100272] some incomplete dg-commands

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |msebor at gcc dot gnu.org

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Martin, you added g++.dg/ext/flexary13.C

This and libffi.complex/complex_int.c is left (libffi is imported from
upstream).

[Bug testsuite/100272] some incomplete dg-commands

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

https://gcc.gnu.org/g:dfdc02bf29670c1c7f5f2820b6db11c66c258716

commit r12-144-gdfdc02bf29670c1c7f5f2820b6db11c66c258716
Author: Richard Biener <rguenther@suse.de>
Date:   Tue Apr 27 11:02:03 2021 +0200

    testsuite/100272 - undo PRE disabling for gcc.dg/tree-ssa/predcom-1.c

    This re-enables PRE and fixes the malformed dg directive pointed
    out in the PR.  It all works as desired and I forgot why I
    disabled this in the past.

    2021-04-27  Richard Biener  <rguenther@suse.de>

            PR testsuite/100272
            * gcc.dg/tree-ssa/predcom-1.c: Re-enable PRE and fix
            malformed dg directive.

[Bug testsuite/100272] some incomplete dg-commands

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|rguenth at gcc dot gnu.org         |unassigned at gcc dot gnu.org
             Status|ASSIGNED                    |NEW

[Bug testsuite/100272] some incomplete dg-commands

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Sebor <msebor@gcc.gnu.org>:

https://gcc.gnu.org/g:2ae2a45c287d254c2890feff2cca46ed2ddb06ca

commit r12-167-g2ae2a45c287d254c2890feff2cca46ed2ddb06ca
Author: Martin Sebor <msebor@redhat.com>
Date:   Tue Apr 27 11:00:53 2021 -0600

    Remove malformed dg-warning directives.

    gcc/testsuite/ChangeLog:
            PR testsuite/100272
            * g++.dg/ext/flexary13.C: Remove malformed directives.

[Bug testsuite/100272] some incomplete dg-commands

[msebor at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100272

Martin Sebor <msebor at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #6 from Martin Sebor <msebor at gcc dot gnu.org> ---
Fixed.