[libcxx] locale.cpp: Move build_name helper into unnamed namespace

[hubert-reinterpretcast]

Fix linkage of build_name; it is not supposed to have external linkage.

[llvmbot]

@llvm/pr-subscribers-libcxx

Author: Hubert Tong (hubert-reinterpretcast)

Changes

Fix linkage of build_name; it is not supposed to have external linkage.

---

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

1 Files Affected:

• (modified) libcxx/src/locale.cpp (+2-2)

diff --git a/libcxx/src/locale.cpp b/libcxx/src/locale.cpp
index 1ca88e30f63ac..c5ab6de5d657a 100644
--- a/libcxx/src/locale.cpp
+++ b/libcxx/src/locale.cpp
@@ -102,8 +102,6 @@ inline constexpr size_t countof(const T* const begin, const T* const end) {
   return static_cast<size_t>(end - begin);
 }
 
-} // namespace
-
 string build_name(const string& other, const string& one, locale::category c) {
   if (other == "*" || one == "*")
     return "*";
@@ -115,6 +113,8 @@ string build_name(const string& other, const string& one, locale::category c) {
   return "*";
 }
 
+} // namespace
+
 const locale::category locale::none;
 const locale::category locale::collate;
 const locale::category locale::ctype;

[github-actions]

⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo.
Please turn off Keep my email addresses private setting in your account.
See LLVM Discourse for more information.

[mordante]

Thanks! LGTM!