Re: [PATCH] backend: compare word-at-a-time in bcTruelen

Jeremy Kerr <jk@ozlabs.org> writes:
> I've put together some data from a microbenchmark of the bcTrulen 
> function, patched and unpatched.

Uh, where's the data?

> In the worst cases, I see a 53% cost increase on x86 (with the string 
> 'aaa ') and a 97% increase on PowerPC ('a  ').
> So, it all depends on the number of padding spaces we'd expect to see on 
> workload data. Fortunately, we see the larger reductions on the more 
> expensive operations (ie, longer strings).

Unfortunately, the cases with lots of padding spaces are probably much
less probable than the cases with fewer.  It would be unpleasant for
example if this patch resulted in a severe performance degradation for
a "canonical" example of char(n) being used properly, such as char(2)
for US state abbreviations.
        regards, tom lane