Re: [PATCH] Only accept BUILT_IN_NORMAL stringops for interesting_stringop_to_profile_p

2015-08-20 Thread Richard Biener
On Thu, Aug 20, 2015 at 11:31 AM, Yangfei (Felix) wrote: > Thanks for the comments. Attached please find the updated patch. OK? Ok. Thanks, Richard. > > Index: gcc/value-prof.c > === > --- gcc/value-prof.c(revision 141081) >

Re: [PATCH] Only accept BUILT_IN_NORMAL stringops for interesting_stringop_to_profile_p

2015-08-20 Thread Yangfei (Felix)
Thanks for the comments. Attached please find the updated patch. OK? Index: gcc/value-prof.c === --- gcc/value-prof.c(revision 141081) +++ gcc/value-prof.c(working copy) @@ -209,7 +209,6 @@ gimple_add_histogram_value (stru

Re: [PATCH] Only accept BUILT_IN_NORMAL stringops for interesting_stringop_to_profile_p

2015-08-20 Thread Richard Biener
On Thu, Aug 20, 2015 at 5:17 AM, Yangfei (Felix) wrote: > Hi, > > As DECL_FUNCTION_CODE is overloaded for builtin functions in different > classes, so need to check builtin class before using fcode. > Patch posted below. Bootstrapped on x86_64-suse-linux, OK for trunk? > Thanks. Ugh

[PATCH] Only accept BUILT_IN_NORMAL stringops for interesting_stringop_to_profile_p

2015-08-19 Thread Yangfei (Felix)
Hi, As DECL_FUNCTION_CODE is overloaded for builtin functions in different classes, so need to check builtin class before using fcode. Patch posted below. Bootstrapped on x86_64-suse-linux, OK for trunk? Thanks. Index: gcc/value-prof.c