[Bug c++/99601] [11 regression] g++.dg/modules/iostream-1_b.C on x86_64 with -m32

2021-03-16 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99601

Richard Biener  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
   Target Milestone|--- |11.0
 Resolution|--- |FIXED

--- Comment #5 from Richard Biener  ---
Fixd.

[Bug c++/99601] [11 regression] g++.dg/modules/iostream-1_b.C on x86_64 with -m32

2021-03-15 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99601

--- Comment #4 from CVS Commits  ---
The master branch has been updated by Jakub Jelinek :

https://gcc.gnu.org/g:1973beaf92c629f26df98eff33ab1bce7f3ea4ef

commit r11-7682-g1973beaf92c629f26df98eff33ab1bce7f3ea4ef
Author: Jakub Jelinek 
Date:   Tue Mar 16 00:24:15 2021 +0100

testsuite: Fix up target selector syntax errors in modules/builtin-3*.C
[PR99601]

Without this patch I'm seeing:
ERROR: tcl error sourcing
/home/jakub/src/gcc/gcc/testsuite/g++.dg/modules/modules.exp.
ERROR: unmatched open brace in list
while executing
"foreach op $tmp {
switch [lindex $op 0] {
"dg-options" {
set std_prefix "-std=gnu++"
if { [string match "*-std=*" [lindex $op 2]] } {
..."
(procedure "module-init" line 7)
invoked from within
"module-init $src"
invoked from within
"if [runtest_file_p $runtests $src] {
set tests [lsort [find [file dirname $src]  [regsub {_a.[CHX]$}
[file tail $src] {_[a-z].[CHX]}]]]

set std_lis..."
("foreach" body line 3)
invoked from within
"foreach src [lsort [find $srcdir/$subdir {*_a.[CHX}]] {
# use the FOO_a.C name as the parallelization key
if [runtest_file_p $runtests $src] {..."
(file "/home/jakub/src/gcc/gcc/testsuite/g++.dg/modules/modules.exp"
line 304)
invoked from within
"source /home/jakub/src/gcc/gcc/testsuite/g++.dg/modules/modules.exp"
("uplevel" body line 1)
invoked from within
"uplevel #0 source
/home/jakub/src/gcc/gcc/testsuite/g++.dg/modules/modules.exp"
invoked from within
"catch "uplevel #0 source $test_file_name""

2021-03-16  Jakub Jelinek  

PR c++/99601
* g++.dg/modules/builtin-3_a.C: Fix target selector syntax errors.
* g++.dg/modules/builtin-3_b.C: Likewise.

[Bug c++/99601] [11 regression] g++.dg/modules/iostream-1_b.C on x86_64 with -m32

2021-03-15 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99601

--- Comment #3 from Uroš Bizjak  ---
(In reply to CVS Commits from comment #1)
> The master branch has been updated by Nathan Sidwell :
> 
> https://gcc.gnu.org/g:770d3487ef18a71f65626c182625889eee29f580
There is a typo in the selector:

+// { dg-final { scan-lang-dump {Wrote GMF:-[0-9]*
type_decl:'::__builtin_va_list'@builtins} module { { target x86_64-*-linux*
i>86-*-linux* } && lp64 } }

i>86-*-linux* should read i?86-*-linux*

[Bug c++/99601] [11 regression] g++.dg/modules/iostream-1_b.C on x86_64 with -m32

2021-03-15 Thread nathan at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99601

--- Comment #2 from Nathan Sidwell  ---
fix the builtin fails:

 770d3487ef1 2021-03-15 | c++: Fix 2 testcases [PR 99601]

I'm not seeing the iostream fail thought

[Bug c++/99601] [11 regression] g++.dg/modules/iostream-1_b.C on x86_64 with -m32

2021-03-15 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99601

--- Comment #1 from CVS Commits  ---
The master branch has been updated by Nathan Sidwell :

https://gcc.gnu.org/g:770d3487ef18a71f65626c182625889eee29f580

commit r11-7679-g770d3487ef18a71f65626c182625889eee29f580
Author: Nathan Sidwell 
Date:   Mon Mar 15 15:25:32 2021 -0400

c++: Fix 2 testcases [PR 99601]

I'd failed to correctly restrict some checks to lp64 x86 targets.

PR c++/99601
gcc/testsuite/
* g++.dg/modules/builtin-3_a.C: Fix lp64 x86 detection.
* g++.dg/modules/builtin-3_b.C: Fix lp64 x86 detection.