Nathan Bossart <nathandboss...@gmail.com> writes: > Here's what I had in mind. My usual benchmark seems to indicate that this > shouldn't impact performance.
Shouldn't "i" be declared uint32, since nelem is? BTW, I wonder why these functions don't declare their array arguments like "const uint32 *base". LGTM otherwise, and I like the fact that the #if structure gets a lot less messy. regards, tom lane