Re: [PATCH] Fix up tree-ssa/strn{cat,cpy-2}.c (PR tree-optimization/83075)

2017-12-08 Thread Jakub Jelinek
On Thu, Dec 07, 2017 at 11:03:10AM -0700, Martin Sebor wrote: > On 12/07/2017 09:55 AM, Jakub Jelinek wrote: > > On Wed, Dec 06, 2017 at 05:30:53PM +0100, Jakub Jelinek wrote: > > > On Wed, Dec 06, 2017 at 09:20:15AM -0700, Martin Sebor wrote: > > > > Attached is a patch with the comment updated/si

Re: [PATCH] Fix up tree-ssa/strn{cat,cpy-2}.c (PR tree-optimization/83075)

2017-12-07 Thread Jeff Law
On 12/07/2017 11:03 AM, Martin Sebor wrote: > On 12/07/2017 09:55 AM, Jakub Jelinek wrote: >> On Wed, Dec 06, 2017 at 05:30:53PM +0100, Jakub Jelinek wrote: >>> On Wed, Dec 06, 2017 at 09:20:15AM -0700, Martin Sebor wrote: Attached is a patch with the comment updated/simplified. The tests

Re: [PATCH] Fix up tree-ssa/strn{cat,cpy-2}.c (PR tree-optimization/83075)

2017-12-07 Thread Martin Sebor
On 12/07/2017 09:55 AM, Jakub Jelinek wrote: On Wed, Dec 06, 2017 at 05:30:53PM +0100, Jakub Jelinek wrote: On Wed, Dec 06, 2017 at 09:20:15AM -0700, Martin Sebor wrote: Attached is a patch with the comment updated/simplified. The tests do the job they need to do today so I just removed the use

Re: [PATCH] Fix up tree-ssa/strn{cat,cpy-2}.c (PR tree-optimization/83075)

2017-12-07 Thread Richard Biener
On December 7, 2017 5:55:47 PM GMT+01:00, Jakub Jelinek wrote: >On Wed, Dec 06, 2017 at 05:30:53PM +0100, Jakub Jelinek wrote: >> On Wed, Dec 06, 2017 at 09:20:15AM -0700, Martin Sebor wrote: >> > Attached is a patch with the comment updated/simplified. >> > The tests do the job they need to do t

[PATCH] Fix up tree-ssa/strn{cat,cpy-2}.c (PR tree-optimization/83075)

2017-12-07 Thread Jakub Jelinek
On Wed, Dec 06, 2017 at 05:30:53PM +0100, Jakub Jelinek wrote: > On Wed, Dec 06, 2017 at 09:20:15AM -0700, Martin Sebor wrote: > > Attached is a patch with the comment updated/simplified. > > The tests do the job they need to do today so I just removed > > the useless attribute but otherwise left t