diff options
| author | Noah Goldstein <goldstein.w.n@gmail.com> | 2022-06-08 21:16:51 -0700 |
|---|---|---|
| committer | Sunil K Pandey <skpgkp2@gmail.com> | 2022-07-18 22:13:57 -0700 |
| commit | b64acbbe2669ef0509b32680b07122740b48cd96 (patch) | |
| tree | 3e58d8895cb064ec5031faa2432839717e6b70aa | |
| parent | a2e10e7a0291b2fe3c67ca9635bc41745e216c95 (diff) | |
| download | glibc-b64acbbe2669ef0509b32680b07122740b48cd96.tar.gz glibc-b64acbbe2669ef0509b32680b07122740b48cd96.tar.xz glibc-b64acbbe2669ef0509b32680b07122740b48cd96.zip | |
x86: Align varshift table to 32-bytes
This ensures the load will never split a cache line. (cherry picked from commit 0f91811333f23b61cf681cab2704b35a0a073b97)
| -rw-r--r-- | sysdeps/x86_64/multiarch/varshift.c | 5 | ||||
| -rw-r--r-- | sysdeps/x86_64/multiarch/varshift.h | 3 |
2 files changed, 5 insertions, 3 deletions
diff --git a/sysdeps/x86_64/multiarch/varshift.c b/sysdeps/x86_64/multiarch/varshift.c index 45267b0a68..1f56354266 100644 --- a/sysdeps/x86_64/multiarch/varshift.c +++ b/sysdeps/x86_64/multiarch/varshift.c @@ -16,9 +16,10 @@ License along with the GNU C Library; if not, see <https://www.gnu.org/licenses/>. */ -#include "varshift.h" +#include <stdint.h> -const int8_t ___m128i_shift_right[31] attribute_hidden = +const int8_t ___m128i_shift_right[31] attribute_hidden + __attribute__((aligned(32))) = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1 diff --git a/sysdeps/x86_64/multiarch/varshift.h b/sysdeps/x86_64/multiarch/varshift.h index 32f2173dd2..745d48fa7c 100644 --- a/sysdeps/x86_64/multiarch/varshift.h +++ b/sysdeps/x86_64/multiarch/varshift.h @@ -19,7 +19,8 @@ #include <stdint.h> #include <tmmintrin.h> -extern const int8_t ___m128i_shift_right[31] attribute_hidden; +extern const int8_t ___m128i_shift_right[31] attribute_hidden + __attribute__ ((aligned (32))); static __inline__ __m128i __m128i_shift_right (__m128i value, unsigned long int offset) |