[llvm] [NFC][LLVM] Namespace cleanup in DeadStoreElimination (PR #163303)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 14 12:50:50 PDT 2025


================
@@ -828,7 +828,7 @@ bool isNoopIntrinsic(Instruction *I) {
 }
 
 // Check if we can ignore \p D for DSE.
-bool canSkipDef(MemoryDef *D, bool DefVisibleToCaller) {
+static bool canSkipDef(MemoryDef *D, bool DefVisibleToCaller) {
----------------
jurahul wrote:

To confirm to LLVM CS: https://llvm.org/docs/CodingStandards.html#restrict-visibility 
> Because of this, we have a simple guideline: make anonymous namespaces as small as possible, and only use them for class declarations

https://github.com/llvm/llvm-project/pull/163303


More information about the llvm-commits mailing list