From: Richard Biener Date: Thu, 27 Mar 2025 10:29:21 +0000 (+0100) Subject: target/119010 - missing reservations for Zen4/5 and SSE compares X-Git-Tag: basepoints/gcc-16~500 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=18668d9c196a36dc7c0cd698a2a767b8f8c8e8a4;p=thirdparty%2Fgcc.git target/119010 - missing reservations for Zen4/5 and SSE compares There's the znver4_sse_test reservation which matches the memory-less SSE compares but currently requires prefix_extra == 1. The old znver automata in this case sometimes uses znver1-double instead of znver1-direct, but it's quite a maze. The following simply drops the prefix_extra requirement, but I have no idea what I'm doing here. There doesn't seem to be any documentation on the scheduler relevant attributes used, or at least I cannot find that. PR target/119010 * config/i386/zn4zn5.md (znver4_sse_test): Drop test of prefix_extra attribute. --- diff --git a/gcc/config/i386/zn4zn5.md b/gcc/config/i386/zn4zn5.md index fb856e9dc98..40e51456a46 100644 --- a/gcc/config/i386/zn4zn5.md +++ b/gcc/config/i386/zn4zn5.md @@ -953,9 +953,8 @@ (define_insn_reservation "znver4_sse_test" 1 (and (eq_attr "cpu" "znver4,znver5") - (and (eq_attr "prefix_extra" "1") - (and (eq_attr "type" "ssecomi") - (eq_attr "memory" "none")))) + (and (eq_attr "type" "ssecomi") + (eq_attr "memory" "none"))) "znver4-direct,znver4-fpu1|znver4-fpu2") (define_insn_reservation "znver4_sse_test_load" 6