[clang] b32a1bd - Revert "[clang][LoongArch] Set MaxAtomicInlineWidth and MaxAtomicPromoteWidth for LoongArch"

Weining Lu via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 11 04:23:01 PDT 2022


Author: Weining Lu
Date: 2022-10-11T19:21:28+08:00
New Revision: b32a1bdf42b0be59d02876874b6764b4700ee7d7

URL: https://github.com/llvm/llvm-project/commit/b32a1bdf42b0be59d02876874b6764b4700ee7d7
DIFF: https://github.com/llvm/llvm-project/commit/b32a1bdf42b0be59d02876874b6764b4700ee7d7.diff

LOG: Revert "[clang][LoongArch] Set MaxAtomicInlineWidth and MaxAtomicPromoteWidth for LoongArch"

This reverts commit 6547565e7bdcd9c3f683ad196b62d08c7061fdf1.

This breaks test: Preprocessor/init-loongarch.c

Added: 
    

Modified: 
    clang/lib/Basic/Targets/LoongArch.h

Removed: 
    clang/test/CodeGen/LoongArch/atomics.c


################################################################################
diff  --git a/clang/lib/Basic/Targets/LoongArch.h b/clang/lib/Basic/Targets/LoongArch.h
index 6a735a7d8f833..5d711c6b1db4c 100644
--- a/clang/lib/Basic/Targets/LoongArch.h
+++ b/clang/lib/Basic/Targets/LoongArch.h
@@ -79,9 +79,6 @@ class LLVM_LIBRARY_VISIBILITY LoongArch32TargetInfo
     }
     return false;
   }
-  void setMaxAtomicWidth() override {
-    MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 32;
-  }
 };
 
 class LLVM_LIBRARY_VISIBILITY LoongArch64TargetInfo
@@ -103,9 +100,6 @@ class LLVM_LIBRARY_VISIBILITY LoongArch64TargetInfo
     }
     return false;
   }
-  void setMaxAtomicWidth() override {
-    MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 64;
-  }
 };
 } // end namespace targets
 } // end namespace clang

diff  --git a/clang/test/CodeGen/LoongArch/atomics.c b/clang/test/CodeGen/LoongArch/atomics.c
deleted file mode 100644
index edc58d30db186..0000000000000
--- a/clang/test/CodeGen/LoongArch/atomics.c
+++ /dev/null
@@ -1,45 +0,0 @@
-// RUN: %clang_cc1 -triple loongarch32 -O1 -emit-llvm %s -o - \
-// RUN:   | FileCheck %s --check-prefix=LA32
-// RUN: %clang_cc1 -triple loongarch64 -O1 -emit-llvm %s -o - \
-// RUN:   | FileCheck %s --check-prefix=LA64
-
-/// This test demonstrates that MaxAtomicInlineWidth is set appropriately.
-
-#include <stdatomic.h>
-#include <stdint.h>
-
-void test_i8_atomics(_Atomic(int8_t) * a, int8_t b) {
-  // LA32: load atomic i8, ptr %a seq_cst, align 1
-  // LA32: store atomic i8 %b, ptr %a seq_cst, align 1
-  // LA32: atomicrmw add ptr %a, i8 %b seq_cst
-  // LA64: load atomic i8, ptr %a seq_cst, align 1
-  // LA64: store atomic i8 %b, ptr %a seq_cst, align 1
-  // LA64: atomicrmw add ptr %a, i8 %b seq_cst
-  __c11_atomic_load(a, memory_order_seq_cst);
-  __c11_atomic_store(a, b, memory_order_seq_cst);
-  __c11_atomic_fetch_add(a, b, memory_order_seq_cst);
-}
-
-void test_i32_atomics(_Atomic(int32_t) * a, int32_t b) {
-  // LA32: load atomic i32, ptr %a seq_cst, align 4
-  // LA32: store atomic i32 %b, ptr %a seq_cst, align 4
-  // LA32: atomicrmw add ptr %a, i32 %b seq_cst
-  // LA64: load atomic i32, ptr %a seq_cst, align 4
-  // LA64: store atomic i32 %b, ptr %a seq_cst, align 4
-  // LA64: atomicrmw add ptr %a, i32 %b seq_cst
-  __c11_atomic_load(a, memory_order_seq_cst);
-  __c11_atomic_store(a, b, memory_order_seq_cst);
-  __c11_atomic_fetch_add(a, b, memory_order_seq_cst);
-}
-
-void test_i64_atomics(_Atomic(int64_t) * a, int64_t b) {
-  // LA32: call i64 @__atomic_load_8
-  // LA32: call void @__atomic_store_8
-  // LA32: call i64 @__atomic_fetch_add_8
-  // LA64: load atomic i64, ptr %a seq_cst, align 8
-  // LA64: store atomic i64 %b, ptr %a seq_cst, align 8
-  // LA64: atomicrmw add ptr %a, i64 %b seq_cst
-  __c11_atomic_load(a, memory_order_seq_cst);
-  __c11_atomic_store(a, b, memory_order_seq_cst);
-  __c11_atomic_fetch_add(a, b, memory_order_seq_cst);
-}


        


More information about the cfe-commits mailing list