[BUILDROBOT] Maybe uninitialized warnings in mips targets (was: [PATCH] Fix PR79345, better uninit warnings for memory)

2017-03-18 Thread Jan-Benedict Glaw
Hi Richard, Catherine, Matthew On Thu, 2017-03-02 14:40:46 +0100, Richard Biener wrote: [...] > On IRC we decided to wait&see for the TREE_NO_WARNING issue. So the > following is what I committed. > > Bootstrapped / tested on x86_64-unknown-linux-gnu. [...] > 2017-03-02 Richard Biener > >

Re: [PATCH] Fix PR79345, better uninit warnings for memory

2017-03-07 Thread Jeff Law
On 03/01/2017 07:03 AM, Richard Biener wrote: The following addresses a regression in uninit warnings that happens because clobber stmts preclude the very simple-minded support we have for memory. The patch fixes this by instead implementing uninit warnings for memory properly, using the alias

Re: [PATCH] Fix PR79345, better uninit warnings for memory

2017-03-02 Thread Richard Biener
On Thu, 2 Mar 2017, Richard Biener wrote: > On Thu, 2 Mar 2017, Jakub Jelinek wrote: > > > On Wed, Mar 01, 2017 at 03:03:29PM +0100, Richard Biener wrote: > > > One issue with the patch is duplicate warnings as TREE_NO_WARNING > > > doesn't work very well on tcc_reference trees which are not > >

Re: [PATCH] Fix PR79345, better uninit warnings for memory

2017-03-02 Thread Richard Biener
On Thu, 2 Mar 2017, Jakub Jelinek wrote: > On Wed, Mar 01, 2017 at 03:03:29PM +0100, Richard Biener wrote: > > One issue with the patch is duplicate warnings as TREE_NO_WARNING > > doesn't work very well on tcc_reference trees which are not > > shared. A followup could use some sort of hash table

Re: [PATCH] Fix PR79345, better uninit warnings for memory

2017-03-02 Thread Jakub Jelinek
On Wed, Mar 01, 2017 at 03:03:29PM +0100, Richard Biener wrote: > One issue with the patch is duplicate warnings as TREE_NO_WARNING > doesn't work very well on tcc_reference trees which are not > shared. A followup could use some sort of hash table to mitigate Can't we use TREE_NO_WARNING on get_

[PATCH] Fix PR79345, better uninit warnings for memory

2017-03-01 Thread Richard Biener
The following addresses a regression in uninit warnings that happens because clobber stmts preclude the very simple-minded support we have for memory. The patch fixes this by instead implementing uninit warnings for memory properly, using the alias oracle walk_aliased_vdefs helper. The patch add