[Bug libstdc++/91653] ostream::operator<<(streambuf*) should fail the ostream when write output stream error but not

2019-09-08 Thread yhliang86 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91653

--- Comment #2 from yinghui  ---
(In reply to Jonathan Wakely from comment #1)
> GCC 4.9.2 has been unsupported for several years.

but both the code and the problem still keeps exactly the same.

I have seen many c++ code write 'os << is.rdbuf()` as a stream copy idiom, and
this may silently write corrupted data in case like disk full.

[Bug target/29998] sh-elf: GCC fails to encode epilogues in unwind-info

2019-09-08 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=29998

Eric Gallager  changed:

   What|Removed |Added

 CC||egallager at gcc dot gnu.org,
   ||pinskia at gcc dot gnu.org
   See Also||https://gcc.gnu.org/bugzill
   ||a/show_bug.cgi?id=12990

--- Comment #2 from Eric Gallager  ---
(In reply to Andrew Pinski from comment #1)
> Again I think this is also a dup of bug 12990 really.

That's been closed as fixed; is this now, too? Putting that under "See Also"
for now until we can confirm it's a dup.

[Bug target/91704] [X86] Codegen for _mm256_cmpgt_epi8 is affected by -funsigned-char

2019-09-08 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91704

Jakub Jelinek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2019-09-08
 CC||jakub at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |jakub at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Jakub Jelinek  ---
Indeed, PR87853 for 32-byte vectors.  I'll handle this.

[Bug target/90698] Darwin X86 backend lacks support for mcmodel={medium, large, kernel}

2019-09-08 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90698

--- Comment #9 from Iain Sandoe  ---
Author: iains
Date: Sun Sep  8 19:49:27 2019
New Revision: 275496

URL: https://gcc.gnu.org/viewcvs?rev=275496=gcc=rev
Log:
[Darwin, X86, testsuite] Add xfails for PR90396.

We don't have support for -mcmodel={medium, large, kernel} so don't
expect tests for those things to work.

For now mark them as xfail where possible and skip where that isn't.
These changes will be logged onto the PR and therefore can be backed
out when the facility is implemented.

2019-09-08  Iain Sandoe  

Backport from mainline.
2019-06-01  Iain Sandoe  

PR target/90698
* gcc.target/i386/pr49866.c: XFAIL for Darwin.
* gcc.target/i386/pr63538.c: Likewise.
* gcc.target/i386/pr61599-1.c: Skip for Darwin.


Modified:
branches/gcc-7-branch/gcc/testsuite/ChangeLog
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr49866.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr61599-1.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr63538.c

[Bug rtl-optimization/64895] RA picks the wrong register for -fipa-ra

2019-09-08 Thread iains at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64895

--- Comment #21 from Iain Sandoe  ---
Author: iains
Date: Sun Sep  8 19:41:20 2019
New Revision: 275495

URL: https://gcc.gnu.org/viewcvs?rev=275495=gcc=rev
Log:
[X86, testsuite] Fix PR rtl-optimisation/64895 XPASSes.

These tests had started to XPASS on pic targets where the codegen
is now as expected.

2019-09-08  Iain Sandoe  

Backport from mainline.
2019-05-23  Iain Sandoe  

PR rtl-optimisation/64895
* gcc.target/i386/fuse-caller-save-rec.c: Remove XFAILs.
* gcc.target/i386/fuse-caller-save.c: Likewise.
* gcc.target/i386/fuse-caller-save-xmm.c: Adjust tests for
PIC cases, remove XFAILs.


Modified:
branches/gcc-7-branch/gcc/testsuite/ChangeLog
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/fuse-caller-save-rec.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/fuse-caller-save-xmm.c
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/fuse-caller-save.c

[Bug target/91704] New: [X86] Codegen for _mm256_cmpgt_epi8 is affected by -funsigned-char

2019-09-08 Thread craig.topper at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91704

Bug ID: 91704
   Summary: [X86] Codegen for _mm256_cmpgt_epi8 is affected by
-funsigned-char
   Product: gcc
   Version: unknown
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: craig.topper at gmail dot com
  Target Milestone: ---

This intrinsic should always do a signed compare, but it uses __v32qi in its
implementation which uses "char" rather than "signed char" in its typedef. This
cause it to follow the -funsigned-char/-fsigned-char setting. The 128-bit
equivalent uses a separate __v16qs typedef to avoid this.

[Bug ada/91703] New: Segmentation Fault

2019-09-08 Thread jordan.mayes21 at hotmail dot co.uk
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91703

Bug ID: 91703
   Summary: Segmentation Fault
   Product: gcc
   Version: unknown
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: ada
  Assignee: unassigned at gcc dot gnu.org
  Reporter: jordan.mayes21 at hotmail dot co.uk
  Target Milestone: ---

Arduino: 1.8.9 (Windows Store 1.8.21.0) (Windows 10), Board: "Arduino/Genuino
Mega or Mega 2560, ATmega2560 (Mega 2560)"

In file included from
C:\Users\Student\Documents\Arduino\sketch_aug09b\sketch_aug09b.ino:2:0:

C:\Users\Student\Documents\Arduino\libraries\FastLED/FastLED.h:14:21: note:
#pragma message: FastLED version 3.003.001

 #pragma message "FastLED version 3.003.001"

 ^

C:\Users\Student\Documents\Arduino\libraries\Elegoo_TFTLCD\Elegoo_TFTLCD.cpp:
In function 'readReg.constprop':

C:\Users\Student\Documents\Arduino\libraries\Elegoo_TFTLCD\Elegoo_TFTLCD.cpp:1019:1:
internal compiler error: Segmentation fault

 }

 ^

Please submit a full bug report,

with preprocessed source if appropriate.

See  for instructions.

lto-wrapper.exe: fatal error: C:\Program
Files\WindowsApps\ArduinoLLC.ArduinoIDE_1.8.21.0_x86__mdqgnx93n4wtt\hardware\tools\avr/bin/avr-gcc
returned 1 exit status

compilation terminated.

c:/program
files/windowsapps/arduinollc.arduinoide_1.8.21.0_x86__mdqgnx93n4wtt/hardware/tools/avr/bin/../lib/gcc/avr/5.4.0/../../../../avr/bin/ld.exe:
error: lto-wrapper failed

collect2.exe: error: ld returned 1 exit status

exit status 1
Error compiling for board Arduino/Genuino Mega or Mega 2560.

This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

[Bug target/91702] New: ICE with mips16

2019-09-08 Thread g...@hauke-m.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91702

Bug ID: 91702
   Summary: ICE with mips16
   Product: gcc
   Version: 9.2.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: target
  Assignee: unassigned at gcc dot gnu.org
  Reporter: g...@hauke-m.de
  Target Milestone: ---

I am getting the following internal compiler error when building json-c-0.13.1
with GCC 9.2 for MIPS big endian with mip16 support.


$ make V=s
make  all-recursive
make[1]: Entering directory '/home/hauke/openwrt/dl/tmp/json-c-0.13.1'
Making all in .
make[2]: Entering directory '/home/hauke/openwrt/dl/tmp/json-c-0.13.1'
/bin/sh ./libtool  --tag=CC   --mode=compile mips-openwrt-linux-gcc
-DHAVE_CONFIG_H -I. -mips16  -Wall -Werror -Wcast-qual
-Wno-error=deprecated-declarations -Wextra -Wno-unused-parameter -D_GNU_SOURCE
-D_REENTRANT -MT json_object.lo -MD -MP -MF .deps/json_object.Tpo -c -o
json_object.lo json_object.c
libtool: compile:  mips-openwrt-linux-gcc -DHAVE_CONFIG_H -I. -mips16 -Wall
-Werror -Wcast-qual -Wno-error=deprecated-declarations -Wextra
-Wno-unused-parameter -D_GNU_SOURCE -D_REENTRANT -MT json_object.lo -MD -MP -MF
.deps/json_object.Tpo -c json_object.c  -fPIC -DPIC -o .libs/json_object.o
during RTL pass: mach
json_object.c: In function 'json_object_int_inc':
json_object.c:735:1: internal compiler error: Segmentation fault
  735 | }
  | ^
0xa1397f crash_signal
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/toplev.c:326
0xcd428e mips_split_move(rtx_def*, rtx_def*, mips_split_type, rtx_def*)
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.c:4871
0xe9cc20 gen_split_59(rtx_insn*, rtx_def**)
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.md:5248
0xeec295 split_3
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.md:5246
0xeec295 split_insns(rtx_def*, rtx_insn*)
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.md:7019
0x72b189 try_split(rtx_def*, rtx_insn*, int)
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/emit-rtl.c:3851
0x9843d1 split_insn
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/recog.c:2901
0x988f67 split_all_insns_noflow()
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/recog.c:3063
0xccb594 mips16_lay_out_constants
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.c:17505
0xcd1765 mips_reorg
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/config/mips/mips.c:19380
0x9b1789 execute
   
/home/hauke/openwrt/openwrt/build_dir/toolchain-mips_24kc_gcc-9.2.0_musl/gcc-9.2.0/gcc/reorg.c:3992
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See  for instructions.
make[2]: *** [Makefile:587: json_object.lo] Error 1
make[2]: Leaving directory '/home/hauke/openwrt/dl/tmp/json-c-0.13.1'
make[1]: *** [Makefile:651: all-recursive] Error 1
make[1]: Leaving directory '/home/hauke/openwrt/dl/tmp/json-c-0.13.1'
make: *** [Makefile:448: all] Error 2
-

The problem is happening in this code part:
https://github.com/json-c/json-c/blob/json-c-0.13.1-20180305/json_object.c#L735

I can reproduce it by using configure with these commands:
CFLAGS="-mips16 " ./configure --target=mips-openwrt-linux
--host=mips-openwrt-linux --build=x86_64-pc-linux-gnu

and then run make

When I remove the -mips16 option from CFLAGS it compiles without an ICE.

I am using GCC 9.2.0 from OpenWrt master as of today with the following patches
applied:
https://git.openwrt.org/?p=openwrt/openwrt.git;a=tree;f=toolchain/gcc/patches/9.2.0;h=81caffc170b7604a685785db350fa5b5e72baf63;hb=HEAD

This was freshly compiled on a Arch Linux system which also uses GCC 9.2.0 as
system compiler.
OpenWrt uses binutils 2.32

I haven't seen this problem with GCC 7.4.0 and GCC 8.3.0, I am also not aware
of any such bug report in OpenWrt.

This is the OpenWrt configuration to reproduce this problem:
--
[hauke@hauke-arch openwrt]$ ./scripts/diffconfig.sh 
CONFIG_TARGET_lantiq=y
CONFIG_TARGET_lantiq_xrx200=y
CONFIG_TARGET_lantiq_xrx200_Default=y
CONFIG_DEVEL=y
CONFIG_TOOLCHAINOPTS=y
# CONFIG_BINUTILS_USE_VERSION_2_31_1 is not set
CONFIG_BINUTILS_USE_VERSION_2_32=y
CONFIG_BINUTILS_VERSION="2.32"
CONFIG_BINUTILS_VERSION_2_32=y
CONFIG_GCC_USE_EMBEDDED_PATH_REMAP=y
# CONFIG_GCC_USE_VERSION_7 is not set
CONFIG_GCC_USE_VERSION_9=y
CONFIG_GCC_VERSION="9.2.0"
CONFIG_GCC_VERSION_9=y

[Bug target/91696] [X86] AVX512 intrinsics that only support SAE should allow (_MM_FOUND_NO_EXC|_MM_FROUND_CUR_DIRECTION) to match icc

2019-09-08 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91696

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #2 from Jakub Jelinek  ---
To me that looks like icc bug that it doesn't sufficiently verify the
arguments, it makes no sense to request two different values to perform the
same thing.

[Bug lto/84579] __gnu_lto_v1 should be removed when linking with -fno-lto

2019-09-08 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84579

--- Comment #9 from Martin Liška  ---
@Romain: Yes, let's close it. Thank you for the testing of the patch in other
branches.

Backporting the patch, please consider also using the following fix:

commit dcaf4c7b4cb9bd70b96b2499ca275d5831328dd0
Author: marxin 
Date:   Tue Aug 27 13:36:15 2019 +

Share a prevailing name for remove debug info symbols w/ LTO.

2019-08-27  Martin Liska  

PR lto/91478
* simple-object-elf.c (simple_object_elf_copy_lto_debug_sections):
First find a WEAK HIDDEN symbol in symbol table that will be
preserved.  Later, use the symbol name for all removed symbols.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@274955
138bc75d-0d04-0410-961f-82ee72b054a4

[Bug c++/91609] friend declaration not honoured

2019-09-08 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91609

Marek Polacek  changed:

   What|Removed |Added

 Status|WAITING |NEW

--- Comment #3 from Marek Polacek  ---
Ah, thanks.  Confirmed thus.

[Bug c++/91701] ICE in condiional noexcept on lambda

2019-09-08 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91701

Marek Polacek  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||mpolacek at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #1 from Marek Polacek  ---
Fixed by the following which is definitely not backportable to 7 at this point,
sorry.

commit 52e76545f1b5c4030498420d8a0c4d6439afb2b9
Author: jason 
Date:   Tue Aug 29 20:37:15 2017 +

Reimplement handling of lambdas in templates.

* cp-tree.h (LAMBDA_FUNCTION_P): Check DECL_DECLARES_FUNCTION_P.
* decl.c (start_preparsed_function): Call start_lambda_scope.
(finish_function): Call finish_lambda_scope.
* init.c (get_nsdmi): Call start/finish_lambda_scope.
* lambda.c (start_lambda_scope): Only ignore VAR_DECL in a
function.
* parser.c (cp_parser_function_definition_after_declarator): Don't
call start/finish_lambda_scope.
* pt.c (retrieve_specialization): Ignore lambda functions in
templates.
(find_parameter_packs_r): Ignore capture proxies.  Look into
lambdas.
(check_for_bare_parameter_packs): Allow bare packs in lambdas.
(tsubst_default_argument): Call start/finish_lambda_scope.
(tsubst_function_decl): Handle lambda functions differently.
(tsubst_template_decl): Likewise.
(tsubst_expr) [DECL_EXPR]: Skip closure declarations and capture
proxies.
(tsubst_lambda_expr): Create a new closure rather than instantiate
the one from the template.
(tsubst_copy_and_build): Don't register a specialization of a pack.
(regenerate_decl_from_template): Call start/finish_lambda_scope.
(instantiate_decl): Remove special lambda function handling.
* semantics.c (process_outer_var_ref): Remove special generic
lambda
handling.  Don't implicitly capture in a lambda in a template. 
Look
for an existing proxy.
* class.c (current_nonlambda_class_type): Use decl_type_context.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@251433
138bc75d-0d04-0410-961f-82ee72b054a4

[Bug other/19180] Improve documentation on How to Add New GCC option

2019-09-08 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19180

Eric Gallager  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Eric Gallager  ---
(In reply to Andrew Pinski from comment #7)
> Things have improved so much since this bug report was written and when I
> confirmed it.  Devang left Apple back in 2012.

OK I guess it can be closed as FIXED then.

[Bug c++/84374] placeholder decltype(auto) accepted when it's not the placeholder alone in trailing-return-type

2019-09-08 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84374

Marek Polacek  changed:

   What|Removed |Added

   Keywords||patch

--- Comment #2 from Marek Polacek  ---
https://gcc.gnu.org/ml/gcc-patches/2019-09/msg00453.html

[Bug c++/85282] CWG 727 (full specialization in non-namespace scope)

2019-09-08 Thread carlo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85282

Carlo Wood  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2019-09-08
 CC||carlo at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #2 from Carlo Wood  ---
I ran into the same problem.

I confirm this problem still exists in the current HEAD (10.0.0),
as well as 9.x and 8.x.

clang compiles the snippet fine.
See my case here: https://wandbox.org/permlink/l5yYYGqsimSQ6Q6M

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Segher Boessenkool  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 CC||segher at gcc dot gnu.org
 Resolution|FIXED   |---

--- Comment #10 from Segher Boessenkool  ---
The new testcases pr91656-[12].c fail on all BE systems as written (the
memmove copies the MSB, not the LSB as intended).

[Bug c++/91701] New: ICE in condiional noexcept on lambda

2019-09-08 Thread gccbugbjorn at fahller dot se
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91701

Bug ID: 91701
   Summary: ICE in condiional noexcept on lambda
   Product: gcc
   Version: 7.4.0
Status: UNCONFIRMED
  Severity: normal
  Priority: P3
 Component: c++
  Assignee: unassigned at gcc dot gnu.org
  Reporter: gccbugbjorn at fahller dot se
  Target Milestone: ---

This bug looks very similar to bug 79590, but that is supposedly fixed in 7.1
and later. This one still fails in 7.1 through 7.4, but works in 8 and 9.

The following code snippet provokes the ICE:

template 
auto convert_to = [](auto v) noexcept(noexcept(T(v)))  { return T(v);};

auto b = convert_to(3);

The message is:

t.cpp: In instantiation of 'struct':
t.cpp:2:6:   required from 'auto convert_to'
t.cpp:4:10:   required from here
t.cpp:2:20: internal compiler error: in nothrow_spec_p, at cp/except.c:1149
 auto convert_to = [](auto v) noexcept(noexcept(T(v)))  { return T(v);};
^
Please submit a full bug report,
with preprocessed source if appropriate.
See  for instructions.
ASM generation compiler returned: 1
t.cpp: In instantiation of 'struct':
t.cpp:2:6:   required from 'auto convert_to'
t.cpp:4:10:   required from here
t.cpp:2:20: internal compiler error: in nothrow_spec_p, at cp/except.c:1149
 auto convert_to = [](auto v) noexcept(noexcept(T(v)))  { return T(v);};
^
Please submit a full bug report,
with preprocessed source if appropriate.
See  for instructions.
Execution build compiler returned: 1

See also: https://gcc.godbolt.org/z/lkoZaT

[Bug target/91683] ICE: SIGSEGV at -O when compiling for riscv64

2019-09-08 Thread zsojka at seznam dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91683

--- Comment #2 from Zdenek Sojka  ---
(In reply to Jim Wilson from comment #1)
> Created attachment 46850 [details]
> proposed fix
> 
> This is failing in undo_all in combine, trying to write to a pointer into
> data that has been garbage collected.
> 
> The problem starts with a riscv.md splitter which calls riscv_move_integer
> which checks can_create_pseudo_p and calls force_reg if allowed.  This
> eventually calls gen_reg_rtx which calls ensure_regno_capacity which calls
> ggc_realloc, which invalidates the data in the undo buffer since all pseudo
> reg addresses change.  I guess this is another reason why calling
> gen_reg_rtx in a splitter is a bad idea.
> 
> I can fix it by adding another option to riscv_move_integer to indicate when
> we are calling it from inside a splitter, but maybe I will think of a better
> way to fix the problem later.
> 

Thank you for the patch!

> The fact that this requires a ggc_realloc call to happen in a very precise
> place means that this testcase does not appear to be very useful.  There is
> no guarantee it will trigger the problem if other parts of the compiler
> change.

Would --param=ggc-min-expand=0 and/or --param=ggc-min-heapsize=0 help to
reproduce the issue?

[Bug testsuite/58321] FAIL: gcc.target/i386/memcpy-strategy-3.c scan-assembler-times memcpy 2 on x86_64-apple-darwin*

2019-09-08 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58321

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #14 from Dominique d'Humieres  ---
Fixed in all open branches, closing.

[Bug pch/61250] Random pch failures with -save-temps on x86_64-apple-darwin1(3-8).

2019-09-08 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61250

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #30 from Dominique d'Humieres  ---
Fixed on all open branches, closing.

[Bug testsuite/67958] The tests changed by r223498 now FAILs on darwin

2019-09-08 Thread dominiq at lps dot ens.fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67958

Dominique d'Humieres  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #12 from Dominique d'Humieres  ---
Fixed in all open branches, closing.

[Bug c++/91609] friend declaration not honoured

2019-09-08 Thread harald at gigawatt dot nl
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91609

Harald van Dijk  changed:

   What|Removed |Added

 CC||harald at gigawatt dot nl

--- Comment #2 from Harald van Dijk  ---
(In reply to Marek Polacek from comment #1)
> clang++ doesn't compile this, either:

I think that's a typo in the test. If `friend result;` is changed to `friend
result_type;`, the example compiles with clang++ but not with g++.

Intel also accepts it. MSVC gets confused and gives a different error which
looks bogus.

A slightly modified example is accepted by clang++, Intel and MSVC, but is
still rejected by g++:

  template 
  struct A {};

  template 
  struct B : A {};

  class C {
  private:
using type = int;
  public:
friend B;
  };

  template struct B;