[llvm] [NFC][LLVM] Cleanup namespace usage in DFAJumpThreading.cpp (PR #162179)

[Rahul Joshi via llvm-commits]

================
@@ -134,11 +133,14 @@ class SelectInstToUnfold {
 
   explicit operator bool() const { return SI && SIUse; }
 };
+} // namespace
 
-void unfold(DomTreeUpdater *DTU, LoopInfo *LI, SelectInstToUnfold SIToUnfold,
-            std::vector<SelectInstToUnfold> *NewSIsToUnfold,
-            std::vector<BasicBlock *> *NewBBs);
+static void unfold(DomTreeUpdater *DTU, LoopInfo *LI,
+                   SelectInstToUnfold SIToUnfold,
+                   std::vector<SelectInstToUnfold> *NewSIsToUnfold,
+                   std::vector<BasicBlock *> *NewBBs);
----------------
jurahul wrote:

Done. I agree that frequent namespace transitions are annoying.

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