For master, let's change the default; see other recent patch.
Committed to gcc-15 branch.

Pietro, thanks again for the patch that led up to this.  This is 
in a different place and with different contents, but you made 
the first move in this direction so IMHO attribution is in order.

-- >8 --
Besides this commit working as a release-branch fix for the
PR, code inspection shows slightly better code for TImode
libgcc functions, and a modified
gcc.c-torture/execute/arith-rand-ll.c (basically s/long
long/__int128 and cutting out the non-128-bit cases) shows a
1.4% improvement.  (Coremark code is identical, as
expected.)

        PR middle-end/120935
        * config/mmix/mmix.h (MAX_FIXED_MODE_SIZE): Define.

Co-authored-by: Pietro Monteiro <pie...@sociotechnical.xyz>
Signed-off-by: Pietro Monteiro <pie...@sociotechnical.xyz>
---
 gcc/config/mmix/mmix.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gcc/config/mmix/mmix.h b/gcc/config/mmix/mmix.h
index 44669e195b4a..354783eccedd 100644
--- a/gcc/config/mmix/mmix.h
+++ b/gcc/config/mmix/mmix.h
@@ -184,6 +184,9 @@ struct GTY(()) machine_function
 /* The lower bits are ignored.  */
 #define STRICT_ALIGNMENT 1
 
+/* With the default, (64 bits), most things still work, but code for
+   e.g. __int128 will be suboptimal. */
+#define MAX_FIXED_MODE_SIZE (2 * BITS_PER_WORD)
 
 /* Node: Type Layout */
 
-- 
2.39.2

Reply via email to