Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-11-05 Thread Jason Merrill
I'm applying this patch again now, this time with a tweak to expmed.c to prevent the false positive warning from breaking bootstrap. I don't see any problem in the gimple output, so the bug must be in the middle-end somewhere. On Thu, Sep 19, 2019 at 8:43 PM Jason Merrill wrote: > > I've reverte

Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-19 Thread Marek Polacek
On Thu, Sep 19, 2019 at 04:11:16PM -0400, Jason Merrill wrote: > Do any of you have a reproducer for this? I've attached it to 91825. Marek

Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-19 Thread Jason Merrill
Do any of you have a reproducer for this? On Thu, Sep 19, 2019 at 3:43 PM Jason Merrill wrote: > > I've reverted this patch for the moment. > > On Wed, Sep 18, 2019 at 8:19 PM JiangNing OS > wrote: > > > > Hi Jason, > > > > This commit caused boot-strap failure on aarch64. Is it a bug? Can this

Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-19 Thread Jason Merrill
I've reverted this patch for the moment. On Wed, Sep 18, 2019 at 8:19 PM JiangNing OS wrote: > > Hi Jason, > > This commit caused boot-strap failure on aarch64. Is it a bug? Can this be > fixed ASAP? > > ../../gcc/gcc/expmed.c:5602:19: error: ���int_mode��� may be used > uninitialized in this f

RE: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-18 Thread JiangNing OS
Hi Jason, This commit caused boot-strap failure on aarch64. Is it a bug? Can this be fixed ASAP? ../../gcc/gcc/expmed.c:5602:19: error: ���int_mode��� may be used uninitialized in this function [-Werror=maybe-uninitialized] 5602 | scalar_int_mode int_mode; | ^~~~

Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-17 Thread Iain Sandoe
> On 17 Sep 2019, at 15:06, Andreas Schwab wrote: > > This breaks bootstrap on aarch64 (during stage2 build): for the record, also on x86_64-darwin1x (attempting some analysis). Iain > > ../../gcc/expmed.c: In function 'rtx_def* emit_store_flag_1(rtx, rtx_code, > rtx, rtx, machine_mode, int

Re: [C++ PATCH 2/4] Fix conversions for built-in operator overloading candidates.

2019-09-17 Thread Andreas Schwab
This breaks bootstrap on aarch64 (during stage2 build): ../../gcc/expmed.c: In function 'rtx_def* emit_store_flag_1(rtx, rtx_code, rtx, rtx, machine_mode, int, int, machine_mode)': ../../gcc/expmed.c:5602:19: error: 'int_mode' may be used uninitialized in this function [-Werror=maybe-uninitializ