Welcome! Log In Create A New Profile

Advanced

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

Maxim Dounin
February 13, 2018 07:32AM
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/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 1 of 2] Optimized murmurhash performance

Matthew Marangoni 528 February 03, 2018 11:42AM

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

Matthew Marangoni 188 February 03, 2018 11:42AM

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

Valentin V. Bartenev 195 February 03, 2018 08:22PM

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

Matthew Marangoni 231 February 12, 2018 03:20PM

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

Maxim Dounin 359 February 13, 2018 07:32AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 237
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready