[PATCH 1/3] amdgcn: Mark s_mulk_i32 as clobbering SCC
Julian Brown
julian@codesourcery.com
Tue Jun 29 14:50:38 GMT 2021
The s_mulk_i32 instruction sets the SCC status register according to
whether the multiplication overflows, but that is not currently modelled
in the GCN backend. AFAIK this is a latent bug and hasn't been noticed
"in the wild", but it should be fixed.
I will commit shortly.
Julian
2021-06-29 Julian Brown <julian@codesourcery.com>
gcc/
* config/gcn/gcn.md (mulsi3): Make s_mulk_i32 variant clobber SCC.
---
gcc/config/gcn/gcn.md | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/gcc/config/gcn/gcn.md b/gcc/config/gcn/gcn.md
index b5f895a93e2..cca45522fba 100644
--- a/gcc/config/gcn/gcn.md
+++ b/gcc/config/gcn/gcn.md
@@ -1371,10 +1371,13 @@
; Vector multiply has vop3a encoding, but no corresponding vop2a, so no long
; immediate.
+; The "s_mulk_i32" variant sets SCC to indicate overflow (which we don't care
+; about here, but we need to indicate the clobbering).
(define_insn "mulsi3"
[(set (match_operand:SI 0 "register_operand" "= Sg,Sg, Sg, v")
(mult:SI (match_operand:SI 1 "gcn_alu_operand" "%SgA, 0,SgA, v")
- (match_operand:SI 2 "gcn_alu_operand" " SgA, J, B,vASv")))]
+ (match_operand:SI 2 "gcn_alu_operand" " SgA, J, B,vASv")))
+ (clobber (match_scratch:BI 3 "=X,cs, X, X"))]
""
"@
s_mul_i32\t%0, %1, %2
--
2.29.2
More information about the Gcc-patches
mailing list