[Bug testsuite/98795] New: modules testsuite testnames

[Bug testsuite/98795] New: modules testsuite testnames

[nathan at gcc dot gnu.org]

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

            Bug ID: 98795
           Summary: modules testsuite testnames
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
          Assignee: unassigned at gcc dot gnu.org
          Reporter: nathan at gcc dot gnu.org
  Target Milestone: ---

[from 98531]

a) header-unit names are from the preprocessor's resolution of the header file.
 We end up with (relative) paths constructed from absolute paths, which is a
problem when comparing test results:

+FAIL: g++.dg/modules/xtreme-header-2_a.H module-cmi 
(gcm.cache/vol/gcc/src/hg/master/local/gcc/testsuite/g++.dg/modules/xtreme-header-2_a.H.gcm)

  must not use absolute filename here

+FAIL: g++.dg/modules/xtreme-header-2_b.C -std=c++2a (test for excess errors)

b) execution tests do not distinguish between c++2a and c++17, so they don't
have unique names.

[Bug testsuite/98795] modules testsuite testnames

[nathan at gcc dot gnu.org]

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

Nathan Sidwell <nathan at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2021-01-22
           Assignee|unassigned at gcc dot gnu.org      |nathan at gcc dot gnu.org

[Bug testsuite/98795] modules testsuite testnames

[cvs-commit at gcc dot gnu.org]

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Nathan Sidwell <nathan@gcc.gnu.org>:

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

commit r11-6860-geee8ed2f22b36dfe64a9516171871476e8ede477
Author: Nathan Sidwell <nathan@acm.org>
Date:   Fri Jan 22 06:44:22 2021 -0800

    testsuite: Uniquify test names [PR 98795]

    Header unit names come from the path the preprocessor determines, and
    thus can be absolute.  This tweaks the testsuite to elide that
    absoluteness when embedded in a CMI name.  We were also not
    distinguishing link and execute tests by the $std flags, so append
    them when necessary.

            PR testsuite/98795
            gcc/testsuite/
            * g++.dg/modules/modules.exp (module_cmi_p): Avoid
            embedded absolute paths.
            (module_do_it): Append $std to test name.

[Bug testsuite/98795] modules testsuite testnames

[nathan at gcc dot gnu.org]

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

Nathan Sidwell <nathan at gcc dot gnu.org> changed:

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

--- Comment #2 from Nathan Sidwell <nathan at gcc dot gnu.org> ---
eee8ed2f22b 2021-01-22 | testsuite: Uniquify test names [PR 98795]