[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