On Wed, 2023-08-09 at 19:46 +0800, Jiajie Chen wrote:
> LoongArch32 only provides basic ll/sc instructions for atomic
> operations. Mark am* atomic instructions as 64-bit only.
I'd prefer using a different symbol, say TARGET_LOONGARCH_AM here. Then
it would be easier to adjust the code if we have a LA32 core with am*
support in the future. For now we can just
#define TARGET_LOONGARCH_AM TARGET_64BIT.
> gcc/ChangeLog:
>
> * config/loongarch.sync.md: Guard am* atomic insns by
> TARGET_64BIT.
> ---
> gcc/config/loongarch/sync.md | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/gcc/config/loongarch/sync.md b/gcc/config/loongarch/sync.md
> index 9924d522bcd..151b553bcc6 100644
> --- a/gcc/config/loongarch/sync.md
> +++ b/gcc/config/loongarch/sync.md
> @@ -77,7 +77,7 @@
> [(match_operand:GPR 1 "reg_or_0_operand" "rJ")
> (match_operand:SI 2 "const_int_operand")] ;; model
> UNSPEC_ATOMIC_STORE))]
> - ""
> + "TARGET_64BIT"
> "amswap%A2.\t$zero,%z1,%0"
> [(set (attr "length") (const_int 8))])
>
> @@ -88,7 +88,7 @@
> (match_operand:GPR 1 "reg_or_0_operand" "rJ"))
> (match_operand:SI 2 "const_int_operand")] ;; model
> UNSPEC_SYNC_OLD_OP))]
> - ""
> + "TARGET_64BIT"
> "am%A2.\t$zero,%z1,%0"
> [(set (attr "length") (const_int 8))])
>
> @@ -101,7 +101,7 @@
> (match_operand:GPR 2 "reg_or_0_operand" "rJ"))
> (match_operand:SI 3 "const_int_operand")] ;; model
> UNSPEC_SYNC_OLD_OP))]
> - ""
> + "TARGET_64BIT"
> "am%A3.\t%0,%z2,%1"
> [(set (attr "length") (const_int 8))])
>
> @@ -113,7 +113,7 @@
> UNSPEC_SYNC_EXCHANGE))
> (set (match_dup 1)
> (match_operand:GPR 2 "register_operand" "r"))]
> - ""
> + "TARGET_64BIT"
> "amswap%A3.\t%0,%z2,%1"
> [(set (attr "length") (const_int 8))])
>
> @@ -182,7 +182,7 @@
> [(match_operand:QI 0 "register_operand" "") ;; bool output
> (match_operand:QI 1 "memory_operand" "+ZB") ;; memory
> (match_operand:SI 2 "const_int_operand" "")] ;; model
> - ""
> + "TARGET_64BIT"
> {
> /* We have no QImode atomics, so use the address LSBs to form a mask,
> then use an aligned SImode atomic. */
--
Xi Ruoyao
School of Aerospace Science and Technology, Xidian University