Re: [PATCH 1/3] Fold BUILT_IN_STRNCASECMP

2016-08-17 Thread Martin Liška
On 08/17/2016 09:10 AM, Jakub Jelinek wrote: > Did you really mean to use this block for strncasecmp only (rather than for > strncmp only, i.e. !is_strncasecmp)? Sure, that was a typo. Unfortunately I had a test case with strings that was eaten by fold_const_call. I enhanced test coverage for that

Re: [PATCH 1/3] Fold BUILT_IN_STRNCASECMP

2016-08-17 Thread Jakub Jelinek
On Tue, Aug 16, 2016 at 03:10:13PM +0200, marxin wrote: > 2016-08-16 Martin Liska > > * builtins.c (fold_builtin_strncmp): Rename to > fold_builtin_strncmp_strncasecmp and support also > strncasecmp. > (fold_builtin_3): Handle BUILT_IN_STRNCASECMP. > --- > -/* Fold funct