Hi, Christophe, I have attached a patch in PR86519, could you please download it and test it, and let me know the result.
thanks. Qing > On Jul 30, 2018, at 8:45 AM, Christophe Lyon <christophe.l...@linaro.org> > wrote: > > On Wed, 25 Jul 2018 at 19:08, Qing Zhao <qing.z...@oracle.com > <mailto:qing.z...@oracle.com>> wrote: >> >> Hi, >> >> As Wilco suggested, the new added strcmp/strncmp inlining should be only >> enabled with O2 and above. >> >> this is the simple patch for this change. >> >> tested on both X86 and aarch64. >> >> Okay for thunk? >> >> Qing >> >> gcc/ChangeLog: >> >> +2018-07-25 Qing Zhao <qing.z...@oracle.com> >> + >> + * builtins.c (inline_expand_builtin_string_cmp): Disable inlining >> + when optimization level is lower than 2 or optimize for size. >> + >> >> gcc/testsuite/ChangeLog: >> >> +2018-07-25 Qing Zhao <qing.z...@oracle.com> >> + >> + * gcc.dg/strcmpopt_5.c: Change to O2 to enable the transformation. >> + * gcc.dg/strcmpopt_6.c: Likewise. >> + >> > > Hi, > > After this change, I've noticed that: > FAIL: gcc.dg/strcmpopt_6.c scan-rtl-dump-times expand "__builtin_memcmp" 6 > on arm-none-linux-gnueabi > --with-mode thumb > --with-cpu cortex-a9 > and forcing -march=armv5t via RUNTESTFLAGS > > The log says: > gcc.dg/strcmpopt_6.c: pattern found 4 times > FAIL: gcc.dg/strcmpopt_6.c scan-rtl-dump-times expand "__builtin_memcmp" 6 > > Christophe