Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-04 Thread Andreas Krebbel via Gcc-patches
On 04.05.20 09:27, Richard Biener wrote: > On Mon, May 4, 2020 at 8:14 AM Andreas Krebbel wrote: >> >> On 30.04.20 18:33, Richard Biener wrote: >>> On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel >>> wrote: On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > On Wed, Apr 29,

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-04 Thread Richard Biener via Gcc-patches
On Mon, May 4, 2020 at 8:14 AM Andreas Krebbel wrote: > > On 30.04.20 18:33, Richard Biener wrote: > > On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel > > wrote: > >> > >> On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > >>> On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: > > >>>

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-05-03 Thread Andreas Krebbel via Gcc-patches
On 30.04.20 18:33, Richard Biener wrote: > On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel wrote: >> >> On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: >>> On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: >

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-30 Thread Richard Biener via Gcc-patches
On Thu, Apr 30, 2020 at 5:14 PM Andreas Krebbel wrote: > > On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > > On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: > >> > >> On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: > >>> > >>> Btw, does s390 have different inli

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-30 Thread Andreas Krebbel via Gcc-patches
On 30.04.20 08:25, Richard Biener via Gcc-patches wrote: > On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: >> >> On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: >>> >>> Btw, does s390 have different inlining parameters somehow? >> I think so. We saw a ton of backend warni

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-29 Thread Richard Biener via Gcc-patches
On Wed, Apr 29, 2020 at 5:56 PM Jeff Law wrote: > > On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: > > > > Btw, does s390 have different inlining parameters somehow? > I think so. We saw a ton of backend warnings that were specific to the s390 > builds in Fedora that app

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-29 Thread Jeff Law via Gcc-patches
On Tue, 2020-04-28 at 11:44 +0200, Richard Biener via Gcc-patches wrote: > > Btw, does s390 have different inlining parameters somehow? I think so. We saw a ton of backend warnings that were specific to the s390 builds in Fedora that appeared to be triggered by different inlining decisions. It h

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-28 Thread Stefan Schulze Frielinghaus via Gcc-patches
On Tue, Apr 28, 2020 at 11:44:58AM +0200, Richard Biener wrote: > On Tue, Apr 28, 2020 at 11:28 AM Stefan Schulze Frielinghaus via > Gcc-patches wrote: > > > > While bootstrapping GCC on S/390 the following warning/error is raised: > > > > gcc/var-tracking.c:10239:34: error: 'pre' may be used unin

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-28 Thread Richard Biener via Gcc-patches
On Tue, Apr 28, 2020 at 11:28 AM Stefan Schulze Frielinghaus via Gcc-patches wrote: > > While bootstrapping GCC on S/390 the following warning/error is raised: > > gcc/var-tracking.c:10239:34: error: 'pre' may be used uninitialized in this > function [-Werror=maybe-uninitialized] > 10239 |

Re: [PATCH] var-tracking.c: Fix possible use of uninitialized variable pre

2020-04-28 Thread Richard Sandiford
Stefan Schulze Frielinghaus via Gcc-patches writes: > While bootstrapping GCC on S/390 the following warning/error is raised: > > gcc/var-tracking.c:10239:34: error: 'pre' may be used uninitialized in this > function [-Werror=maybe-uninitialized] > 10239 | VTI (bb)->out.stack_adjust += pre;