diff mbox series

math: Improve layout of expf data

Message ID PAWPR08MB8982B598CD5FCBC524BAE52C83B72@PAWPR08MB8982.eurprd08.prod.outlook.com
State New
Headers show
Series math: Improve layout of expf data | expand

Commit Message

Wilco Dijkstra July 29, 2024, 2:15 p.m. UTC 
Move the shift field after poly_scaled to ensure load-pair instructions
are aligned on targets that support them.

OK for commit?

---

Comments

Alexander Monakov July 29, 2024, 2:24 p.m. UTC | #1 
On Mon, 29 Jul 2024, Wilco Dijkstra wrote:

> Move the shift field after poly_scaled to ensure load-pair instructions
> are aligned on targets that support them.
> 
> OK for commit?

I cannot tell which fields are supposed to be aligned after your patch
(neither from your text, nor from the diff itself). I also don't see
how alignment would change, considering the struct itself carries no
additional alignment attribute.

Alexander
diff mbox series

Patch

diff --git a/sysdeps/ieee754/flt-32/math_config.h b/sysdeps/ieee754/flt-32/math_config.h
index 729f22cd4f7dd9e44ce5e97fb6a399e29e37eb13..dc07ebd45977e5118a12c6bb7c738d646b0f7926 100644
--- a/sysdeps/ieee754/flt-32/math_config.h
+++ b/sysdeps/ieee754/flt-32/math_config.h
@@ -166,9 +166,9 @@  extern const struct exp2f_data
   uint64_t tab[1 << EXP2F_TABLE_BITS];
   double shift_scaled;
   double poly[EXP2F_POLY_ORDER];
-  double shift;
   double invln2_scaled;
   double poly_scaled[EXP2F_POLY_ORDER];
+  double shift;
 } __exp2f_data attribute_hidden;
 
 #define LOGF_TABLE_BITS 4