Re: Fix handling of stores in modref_summary::useful_p

2020-10-06 Thread Szabolcs Nagy via Gcc-patches
The 10/05/2020 23:45, Jan Hubicka wrote: > > The 10/05/2020 17:28, Szabolcs Nagy via Gcc-patches wrote: > > minimal reproducer: > > > > #include > > int main() > > { > > int r,t; > > r = sscanf("01", "%2x", ); > > printf("scanf: %d %02x\n", r, t); > > return 0; >

Re: Fix handling of stores in modref_summary::useful_p

2020-10-05 Thread Jan Hubicka
> The 10/05/2020 17:28, Szabolcs Nagy via Gcc-patches wrote: > > The 10/05/2020 12:52, Vaseeharan Vinayagamoorthy wrote: > > > Hi, > > > > > > After this patch, I am noticing that some glibc crypto tests get stuck in > > > scanf which goes into busy loop. > > > > > > My build/host/target setup

Re: Fix handling of stores in modref_summary::useful_p

2020-10-05 Thread Szabolcs Nagy via Gcc-patches
The 10/05/2020 17:28, Szabolcs Nagy via Gcc-patches wrote: > The 10/05/2020 12:52, Vaseeharan Vinayagamoorthy wrote: > > Hi, > > > > After this patch, I am noticing that some glibc crypto tests get stuck in > > scanf which goes into busy loop. > > > > My build/host/target setup is: > > Build:

Re: Fix handling of stores in modref_summary::useful_p

2020-10-05 Thread Szabolcs Nagy via Gcc-patches
The 10/05/2020 12:52, Vaseeharan Vinayagamoorthy wrote: > Hi, > > After this patch, I am noticing that some glibc crypto tests get stuck in > scanf which goes into busy loop. > > My build/host/target setup is: > Build: aarch64-none-linux-gnu > Host: aarch64-none-linux-gnu > Target:

Re: Fix handling of stores in modref_summary::useful_p

2020-10-05 Thread Vaseeharan Vinayagamoorthy via Gcc-patches
Hi, After this patch, I am noticing that some glibc crypto tests get stuck in scanf which goes into busy loop. My build/host/target setup is: Build: aarch64-none-linux-gnu Host: aarch64-none-linux-gnu Target: aarch64-none-linux-gnu Kind regards Vasee On 27/09/2020, 22:46, "Gcc-patches on

Fix handling of stores in modref_summary::useful_p

2020-09-27 Thread Jan Hubicka
Hi, this patch fixes a pasto in modref_summary::useful_p that made ipa-modref to give up on tracking stores when all load info got lost. Bootstrapped/regtested x86_64-linux, comitted. gcc/ChangeLog: 2020-09-27 Jan Hubicka * ipa-modref.c (modref_summary::useful_p): Fix testing of