Re: [PATCH] Add memmove to value profiling (PR value-prof/35543).

2018-08-01 Thread Martin Liška
On 08/01/2018 12:14 PM, Jan Hubicka wrote: > OK, thanks! > We have other builtins that may fold into string function which we expand > internally (str variants comes to mind) perhaps they could be instrumented, > too. Sure, as mentioned here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=35543#c5,

Re: [PATCH] Add memmove to value profiling (PR value-prof/35543).

2018-08-01 Thread Jan Hubicka
> Hi. > > As requested in the PR, I would like to add value profiling for > BUILT_IN_MEMMOVE. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Martin > > gcc/ChangeLog: > > 2018-07-31 Martin Liska > > PR value-prof/35543 >

[PATCH] Add memmove to value profiling (PR value-prof/35543).

2018-08-01 Thread Martin Liška
Hi. As requested in the PR, I would like to add value profiling for BUILT_IN_MEMMOVE. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2018-07-31 Martin Liska PR value-prof/35543 * value-prof.c (interesting_