chore: Remove USE_ALIGNED_ACCESS and enhance BYTE_ORDER handling
Redis uses lot of USE_ALIGNED_ACCESS as the "fastpath" for ARM like archtecture, however, I think modern compiler can handle this kind of optimization well. So this part of code is removed.
Besides, some vendored libraray uses BYTE_ORDER macro, this might not being defined in these files. So I use BYTE_ORDER instead
@git-hulk @PragmaTwice I've paste the result https://github.com/apache/kvrocks/pull/2456#discussion_r1702804569
https://github.com/apache/arrow/blob/45b176716cc667384577a2a1218c6da454854109/cpp/src/arrow/util/bit_util_benchmark.cc#L165-L189
The code runs same speed with highly optimized code in macos, and x86 would share this optimization
Quality Gate passed
Issues
4 New issues
0 Accepted issues
Measures
0 Security Hotspots
59.6% Coverage on New Code
1.0% Duplication on New Code
@PragmaTwice would you mind check again?