Re: PR80613

2017-05-05 Thread Richard Biener
t;p = __builtin_strdup (t); > >> >p = __builtin_strdup (t); > >> >return p; > >> > } > >> > > >> > That's fixed in patch by adding strdup/strndup to another > >> > corresponding condition in propagate_necessity() so that only one &

Re: PR80613

2017-05-05 Thread Prathamesh Kulkarni
up/strndup to another >> > corresponding condition in propagate_necessity() so that only one >> > instance of strdup would be kept. >> > >> > Bootstrapped+tested on x86_64-unknown-linux-gnu. >> > Cross-testing on arm*-*-* and aarch64*-*-* in

Re: PR80613

2017-05-05 Thread Richard Biener
e kept. > > > > Bootstrapped+tested on x86_64-unknown-linux-gnu. > > Cross-testing on arm*-*-* and aarch64*-*-* in progress. > > OK to commit if testing passes ? > > > > Thanks > > Prathamesh > > > > > > pr80613-1.txt > > > > > > 2017-

Re: PR80613

2017-05-04 Thread Jeff Law
ped+tested on x86_64-unknown-linux-gnu. Cross-testing on arm*-*-* and aarch64*-*-* in progress. OK to commit if testing passes ? Thanks Prathamesh pr80613-1.txt 2017-05-04 Prathamesh Kulkarni PR tree-optimization/80613 * tree-ssa-dce.c (propagate_necessity): Add cas

Re: PR80613

2017-05-04 Thread Jeff Law
On 05/04/2017 11:52 AM, Martin Sebor wrote: On 05/04/2017 10:00 AM, Prathamesh Kulkarni wrote: Hi, As mentioned in PR, the issue is that cddce1 marks the call to __builtin_strdup as necessary: marking necessary through .MEM_6 stmt p_7 = __builtin_strdup (&d); and since p_7 doesn't get added to

Re: PR80613

2017-05-04 Thread Trevor Saunders
On Thu, May 04, 2017 at 11:52:31AM -0600, Martin Sebor wrote: > On 05/04/2017 10:00 AM, Prathamesh Kulkarni wrote: > > Hi, > > As mentioned in PR, the issue is that cddce1 marks the call to > > __builtin_strdup as necessary: > > marking necessary through .MEM_6 stmt p_7 = __builtin_strdup (&d); > >

Re: PR80613

2017-05-04 Thread Martin Sebor
On 05/04/2017 10:00 AM, Prathamesh Kulkarni wrote: Hi, As mentioned in PR, the issue is that cddce1 marks the call to __builtin_strdup as necessary: marking necessary through .MEM_6 stmt p_7 = __builtin_strdup (&d); and since p_7 doesn't get added to worklist in propagate_necessity() because it'

PR80613

2017-05-04 Thread Prathamesh Kulkarni
ssary_stmts): Likewise. testsuite/ * gcc.dg/tree-ssa/pr80613-1.c: New test-case. * gcc.dg/tree-ssa/pr80613-2.c: New test-case. diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr80613-1.c b/gcc/testsuite/gcc.dg/tree-ssa/pr80613-1.c new file mode 100644 index 000..56176427922 --- /d