[PATCH 2 of 2] Updated __ORDER_LITTLE_ENDIAN__ to match NGX standards and changed to uint32_t data type

Maxim Dounin mdounin at mdounin.ru
Tue Feb 13 12:30:32 UTC 2018


Hello!

On Mon, Feb 12, 2018 at 03:18:34PM -0500, Matthew Marangoni wrote:

> Are there any existing macro's that determine if the platform has alignment
> resolution issues?

There is, usual check is "#if (NGX_HAVE_LITTLE_ENDIAN && 
NGX_HAVE_NONALIGNED)", take a look at src/core/ngx_md5.c or for an 
example.

On the other hand, I would rather recommend to avoid such 
micro-optimizations unless you have a really good reason for them 
(read: unless you have a real-world use case where the change 
results in a statistically significant performance improvement).

[...]

-- 
Maxim Dounin
http://mdounin.ru/


More information about the nginx-devel mailing list