[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-14 Thread via cfe-commits 


@@ -517,13 +517,13 @@ bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
 Changed |= !CastsCache.empty();
   }
   // Merge all globals within same address space into single
-  // .arena. section
+  // .address_space. section
   for (GlobalVariable &G : M.globals()) {
 if (G.getAddressSpace() == 0 || G.hasSection())
   continue;
 SmallString<16> SecName;
 raw_svector_ostream OS(SecName);
-OS << ".arena." << G.getAddressSpace();
+OS << ".address_space." << G.getAddressSpace();

eddyz87 wrote:

Renamed all "address_space" to "addr_space".

https://github.com/llvm/llvm-project/pull/85161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-14 Thread via cfe-commits 

https://github.com/eddyz87 updated 
https://github.com/llvm/llvm-project/pull/85161

>From ab31efc46153f7065b186b37e406d72188c4f780 Mon Sep 17 00:00:00 2001
From: Eduard Zingerman 
Date: Thu, 14 Mar 2024 01:56:18 +0200
Subject: [PATCH 1/2] [BPF] rename 'arena' to 'address_space'

There are a few places where 'arena' name is used for pointers in
non-zero address space in BPF backend, rename these to use a more
generic 'address_space':
- macro __BPF_FEATURE_ARENA_CAST -> __BPF_FEATURE_ADDRESS_SPACE_CAST
- name for arena global variables section .arena.N -> .address_space.N
---
 clang/lib/Basic/Targets/BPF.cpp |  2 +-
 clang/test/Preprocessor/bpf-predefined-macros.c |  2 +-
 llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp | 10 +-
 llvm/test/CodeGen/BPF/addr-space-globals.ll |  2 +-
 llvm/test/CodeGen/BPF/addr-space-globals2.ll|  4 ++--
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/clang/lib/Basic/Targets/BPF.cpp b/clang/lib/Basic/Targets/BPF.cpp
index 26a54f631fcfc4..9a1188acab944c 100644
--- a/clang/lib/Basic/Targets/BPF.cpp
+++ b/clang/lib/Basic/Targets/BPF.cpp
@@ -36,7 +36,7 @@ void BPFTargetInfo::getTargetDefines(const LangOptions &Opts,
 return;
   }
 
-  Builder.defineMacro("__BPF_FEATURE_ARENA_CAST");
+  Builder.defineMacro("__BPF_FEATURE_ADDRESS_SPACE_CAST");
 
   if (CPU.empty() || CPU == "generic" || CPU == "v1") {
 Builder.defineMacro("__BPF_CPU_VERSION__", "1");
diff --git a/clang/test/Preprocessor/bpf-predefined-macros.c 
b/clang/test/Preprocessor/bpf-predefined-macros.c
index fea24d1ea0ff7b..b6928c5c6ff4b4 100644
--- a/clang/test/Preprocessor/bpf-predefined-macros.c
+++ b/clang/test/Preprocessor/bpf-predefined-macros.c
@@ -61,7 +61,7 @@ int r;
 #ifdef __BPF_FEATURE_ST
 int s;
 #endif
-#ifdef __BPF_FEATURE_ARENA_CAST
+#ifdef __BPF_FEATURE_ADDRESS_SPACE_CAST
 int t;
 #endif
 
diff --git a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp 
b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
index edd59aaa6d01d2..070dd1ff4c5784 100644
--- a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
+++ b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
@@ -15,7 +15,7 @@
 //   - remove llvm.bpf.getelementptr.and.load builtins.
 //   - remove llvm.bpf.getelementptr.and.store builtins.
 //   - for loads and stores with base addresses from non-zero address space
-// cast base address to zero address space (support for BPF arenas).
+// cast base address to zero address space (support for BPF address 
spaces).
 //
 
//===--===//
 
@@ -482,7 +482,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
   }
 }
 
-// Support for BPF arenas:
+// Support for BPF address spaces:
 // - for each function in the module M, update pointer operand of
 //   each memory access instruction (load/store/cmpxchg/atomicrmw)
 //   by casting it from non-zero address space to zero address space, e.g:
@@ -490,7 +490,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
 //   (load (ptr addrspace (N) %p) ...)
 // -> (load (addrspacecast ptr addrspace (N) %p to ptr))
 //
-// - assign section with name .arena.N for globals defined in
+// - assign section with name .address_space.N for globals defined in
 //   non-zero address space N
 bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
   bool Changed = false;
@@ -517,13 +517,13 @@ bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
 Changed |= !CastsCache.empty();
   }
   // Merge all globals within same address space into single
-  // .arena. section
+  // .address_space. section
   for (GlobalVariable &G : M.globals()) {
 if (G.getAddressSpace() == 0 || G.hasSection())
   continue;
 SmallString<16> SecName;
 raw_svector_ostream OS(SecName);
-OS << ".arena." << G.getAddressSpace();
+OS << ".address_space." << G.getAddressSpace();
 G.setSection(SecName);
 // Prevent having separate section for constants
 G.setConstant(false);
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals.ll
index 878ba0dfce6cd1..9848b1bbcd4f35 100644
--- a/llvm/test/CodeGen/BPF/addr-space-globals.ll
+++ b/llvm/test/CodeGen/BPF/addr-space-globals.ll
@@ -18,7 +18,7 @@
 
 ; Verify that a,b,c reside in the same section
 
-; CHECK: .section .arena.272,"aw",@progbits
+; CHECK: .section .address_space.272,"aw",@progbits
 ; CHECK-NOT: .section
 ; CHECK: .globl  a
 ; CHECK: .ascii  "\001\002"
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals2.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals2.ll
index d1e2318948751e..816741e2e834eb 100644
--- a/llvm/test/CodeGen/BPF/addr-space-globals2.ll
+++ b/llvm/test/CodeGen/BPF/addr-space-globals2.ll
@@ -14,12 +14,12 @@
 
 ; Verify that a,b reside in separate sections
 
-; CHECK: .section .arena.1,"aw",@progbits
+; CHECK: .section .address_space.1,"aw",@progbits
 ; CHECK-NOT: .section
 ; CHECK: .globl  a
 ; CHECK:

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 


@@ -517,13 +517,13 @@ bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
 Changed |= !CastsCache.empty();
   }
   // Merge all globals within same address space into single
-  // .arena. section
+  // .address_space. section
   for (GlobalVariable &G : M.globals()) {
 if (G.getAddressSpace() == 0 || G.hasSection())
   continue;
 SmallString<16> SecName;
 raw_svector_ostream OS(SecName);
-OS << ".arena." << G.getAddressSpace();
+OS << ".address_space." << G.getAddressSpace();

4ast wrote:

since instruction is called "addr_space" let's name the section name the same 
".addr_space." ?
and __BPF_FEATURE_ADDR_SPACE_CAST too ?

https://github.com/llvm/llvm-project/pull/85161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 

https://github.com/4ast requested changes to this pull request.


https://github.com/llvm/llvm-project/pull/85161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 

https://github.com/4ast edited https://github.com/llvm/llvm-project/pull/85161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 

eddyz87 wrote:

I also tested this change using 
[this](https://gist.github.com/eddyz87/1921247819138a9ed70d77db40db8a7c) kernel 
patch and arena tests are passing.

https://github.com/llvm/llvm-project/pull/85161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 

llvmbot wrote:




@llvm/pr-subscribers-clang

Author: None (eddyz87)


Changes

There are a few places where `arena` name is used for pointers in non-zero 
address space in BPF backend, rename these to use a more generic 
`address_space`:
- macro `__BPF_FEATURE_ARENA_CAST` -> `__BPF_FEATURE_ADDRESS_SPACE_CAST
- name for arena global variables section `.arena.N` -> `.address_space.N`

---
Full diff: https://github.com/llvm/llvm-project/pull/85161.diff


5 Files Affected:

- (modified) clang/lib/Basic/Targets/BPF.cpp (+1-1) 
- (modified) clang/test/Preprocessor/bpf-predefined-macros.c (+1-1) 
- (modified) llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp (+5-5) 
- (modified) llvm/test/CodeGen/BPF/addr-space-globals.ll (+1-1) 
- (modified) llvm/test/CodeGen/BPF/addr-space-globals2.ll (+2-2) 


``diff
diff --git a/clang/lib/Basic/Targets/BPF.cpp b/clang/lib/Basic/Targets/BPF.cpp
index 26a54f631fcfc4..9a1188acab944c 100644
--- a/clang/lib/Basic/Targets/BPF.cpp
+++ b/clang/lib/Basic/Targets/BPF.cpp
@@ -36,7 +36,7 @@ void BPFTargetInfo::getTargetDefines(const LangOptions &Opts,
 return;
   }
 
-  Builder.defineMacro("__BPF_FEATURE_ARENA_CAST");
+  Builder.defineMacro("__BPF_FEATURE_ADDRESS_SPACE_CAST");
 
   if (CPU.empty() || CPU == "generic" || CPU == "v1") {
 Builder.defineMacro("__BPF_CPU_VERSION__", "1");
diff --git a/clang/test/Preprocessor/bpf-predefined-macros.c 
b/clang/test/Preprocessor/bpf-predefined-macros.c
index fea24d1ea0ff7b..b6928c5c6ff4b4 100644
--- a/clang/test/Preprocessor/bpf-predefined-macros.c
+++ b/clang/test/Preprocessor/bpf-predefined-macros.c
@@ -61,7 +61,7 @@ int r;
 #ifdef __BPF_FEATURE_ST
 int s;
 #endif
-#ifdef __BPF_FEATURE_ARENA_CAST
+#ifdef __BPF_FEATURE_ADDRESS_SPACE_CAST
 int t;
 #endif
 
diff --git a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp 
b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
index edd59aaa6d01d2..070dd1ff4c5784 100644
--- a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
+++ b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
@@ -15,7 +15,7 @@
 //   - remove llvm.bpf.getelementptr.and.load builtins.
 //   - remove llvm.bpf.getelementptr.and.store builtins.
 //   - for loads and stores with base addresses from non-zero address space
-// cast base address to zero address space (support for BPF arenas).
+// cast base address to zero address space (support for BPF address 
spaces).
 //
 
//===--===//
 
@@ -482,7 +482,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
   }
 }
 
-// Support for BPF arenas:
+// Support for BPF address spaces:
 // - for each function in the module M, update pointer operand of
 //   each memory access instruction (load/store/cmpxchg/atomicrmw)
 //   by casting it from non-zero address space to zero address space, e.g:
@@ -490,7 +490,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
 //   (load (ptr addrspace (N) %p) ...)
 // -> (load (addrspacecast ptr addrspace (N) %p to ptr))
 //
-// - assign section with name .arena.N for globals defined in
+// - assign section with name .address_space.N for globals defined in
 //   non-zero address space N
 bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
   bool Changed = false;
@@ -517,13 +517,13 @@ bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
 Changed |= !CastsCache.empty();
   }
   // Merge all globals within same address space into single
-  // .arena. section
+  // .address_space. section
   for (GlobalVariable &G : M.globals()) {
 if (G.getAddressSpace() == 0 || G.hasSection())
   continue;
 SmallString<16> SecName;
 raw_svector_ostream OS(SecName);
-OS << ".arena." << G.getAddressSpace();
+OS << ".address_space." << G.getAddressSpace();
 G.setSection(SecName);
 // Prevent having separate section for constants
 G.setConstant(false);
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals.ll
index 878ba0dfce6cd1..9848b1bbcd4f35 100644
--- a/llvm/test/CodeGen/BPF/addr-space-globals.ll
+++ b/llvm/test/CodeGen/BPF/addr-space-globals.ll
@@ -18,7 +18,7 @@
 
 ; Verify that a,b,c reside in the same section
 
-; CHECK: .section .arena.272,"aw",@progbits
+; CHECK: .section .address_space.272,"aw",@progbits
 ; CHECK-NOT: .section
 ; CHECK: .globl  a
 ; CHECK: .ascii  "\001\002"
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals2.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals2.ll
index d1e2318948751e..816741e2e834eb 100644
--- a/llvm/test/CodeGen/BPF/addr-space-globals2.ll
+++ b/llvm/test/CodeGen/BPF/addr-space-globals2.ll
@@ -14,12 +14,12 @@
 
 ; Verify that a,b reside in separate sections
 
-; CHECK: .section .arena.1,"aw",@progbits
+; CHECK: .section .address_space.1,"aw",@progbits
 ; CHECK-NOT: .section
 ; CHECK: .globl  a
 ; CHECK: .ascii  "\001\002"
 
-; CHECK: .section .arena.2,"aw",@progbits
+; CHECK: .section .address_space.2,"aw",@pr

[clang] [llvm] [BPF] rename 'arena' to 'address_space' (PR #85161)

2024-03-13 Thread via cfe-commits 

https://github.com/eddyz87 created 
https://github.com/llvm/llvm-project/pull/85161

There are a few places where `arena` name is used for pointers in non-zero 
address space in BPF backend, rename these to use a more generic 
`address_space`:
- macro `__BPF_FEATURE_ARENA_CAST` -> `__BPF_FEATURE_ADDRESS_SPACE_CAST
- name for arena global variables section `.arena.N` -> `.address_space.N`

>From d309957bc26f3f9d4afba6a1d0b6bd3ddffbfee7 Mon Sep 17 00:00:00 2001
From: Eduard Zingerman 
Date: Thu, 14 Mar 2024 01:56:18 +0200
Subject: [PATCH] [BPF] rename 'arena' to 'address_space'

There are a few places where 'arena' name is used for pointers in
non-zero address space in BPF backend, rename these to use a more
generic 'address_space':
- macro __BPF_FEATURE_ARENA_CAST -> __BPF_FEATURE_ADDRESS_SPACE_CAST
- name for arena global variables section .arena.N -> .address_space.N
---
 clang/lib/Basic/Targets/BPF.cpp |  2 +-
 clang/test/Preprocessor/bpf-predefined-macros.c |  2 +-
 llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp | 10 +-
 llvm/test/CodeGen/BPF/addr-space-globals.ll |  2 +-
 llvm/test/CodeGen/BPF/addr-space-globals2.ll|  4 ++--
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/clang/lib/Basic/Targets/BPF.cpp b/clang/lib/Basic/Targets/BPF.cpp
index 26a54f631fcfc4..9a1188acab944c 100644
--- a/clang/lib/Basic/Targets/BPF.cpp
+++ b/clang/lib/Basic/Targets/BPF.cpp
@@ -36,7 +36,7 @@ void BPFTargetInfo::getTargetDefines(const LangOptions &Opts,
 return;
   }
 
-  Builder.defineMacro("__BPF_FEATURE_ARENA_CAST");
+  Builder.defineMacro("__BPF_FEATURE_ADDRESS_SPACE_CAST");
 
   if (CPU.empty() || CPU == "generic" || CPU == "v1") {
 Builder.defineMacro("__BPF_CPU_VERSION__", "1");
diff --git a/clang/test/Preprocessor/bpf-predefined-macros.c 
b/clang/test/Preprocessor/bpf-predefined-macros.c
index fea24d1ea0ff7b..b6928c5c6ff4b4 100644
--- a/clang/test/Preprocessor/bpf-predefined-macros.c
+++ b/clang/test/Preprocessor/bpf-predefined-macros.c
@@ -61,7 +61,7 @@ int r;
 #ifdef __BPF_FEATURE_ST
 int s;
 #endif
-#ifdef __BPF_FEATURE_ARENA_CAST
+#ifdef __BPF_FEATURE_ADDRESS_SPACE_CAST
 int t;
 #endif
 
diff --git a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp 
b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
index edd59aaa6d01d2..070dd1ff4c5784 100644
--- a/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
+++ b/llvm/lib/Target/BPF/BPFCheckAndAdjustIR.cpp
@@ -15,7 +15,7 @@
 //   - remove llvm.bpf.getelementptr.and.load builtins.
 //   - remove llvm.bpf.getelementptr.and.store builtins.
 //   - for loads and stores with base addresses from non-zero address space
-// cast base address to zero address space (support for BPF arenas).
+// cast base address to zero address space (support for BPF address 
spaces).
 //
 
//===--===//
 
@@ -482,7 +482,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
   }
 }
 
-// Support for BPF arenas:
+// Support for BPF address spaces:
 // - for each function in the module M, update pointer operand of
 //   each memory access instruction (load/store/cmpxchg/atomicrmw)
 //   by casting it from non-zero address space to zero address space, e.g:
@@ -490,7 +490,7 @@ static void aspaceWrapOperand(DenseMap 
&Cache, Instruction *I,
 //   (load (ptr addrspace (N) %p) ...)
 // -> (load (addrspacecast ptr addrspace (N) %p to ptr))
 //
-// - assign section with name .arena.N for globals defined in
+// - assign section with name .address_space.N for globals defined in
 //   non-zero address space N
 bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
   bool Changed = false;
@@ -517,13 +517,13 @@ bool BPFCheckAndAdjustIR::insertASpaceCasts(Module &M) {
 Changed |= !CastsCache.empty();
   }
   // Merge all globals within same address space into single
-  // .arena. section
+  // .address_space. section
   for (GlobalVariable &G : M.globals()) {
 if (G.getAddressSpace() == 0 || G.hasSection())
   continue;
 SmallString<16> SecName;
 raw_svector_ostream OS(SecName);
-OS << ".arena." << G.getAddressSpace();
+OS << ".address_space." << G.getAddressSpace();
 G.setSection(SecName);
 // Prevent having separate section for constants
 G.setConstant(false);
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals.ll
index 878ba0dfce6cd1..9848b1bbcd4f35 100644
--- a/llvm/test/CodeGen/BPF/addr-space-globals.ll
+++ b/llvm/test/CodeGen/BPF/addr-space-globals.ll
@@ -18,7 +18,7 @@
 
 ; Verify that a,b,c reside in the same section
 
-; CHECK: .section .arena.272,"aw",@progbits
+; CHECK: .section .address_space.272,"aw",@progbits
 ; CHECK-NOT: .section
 ; CHECK: .globl  a
 ; CHECK: .ascii  "\001\002"
diff --git a/llvm/test/CodeGen/BPF/addr-space-globals2.ll 
b/llvm/test/CodeGen/BPF/addr-space-globals2.ll
index d1e2318948751e..816741e2e834eb 100644
--- a/llvm/test/CodeGen/BPF/add