[libc-commits] [libc] [libc] Fix -Wshadow warning in sqrtf128.h (PR #196851)

via libc-commits libc-commits at lists.llvm.org
Sun May 10 17:37:36 PDT 2026


llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: Nico Weber (nico)

<details>
<summary>Changes</summary>

sqrtf128() contained both `using namespace sqrtf128_internal;` and `using FPBits = fputil::FPBits<float128>;`, but sqrtf128_internal also had a `using FPBits = fputil::FPBits<float128>;`. The outer `using` wasn't actually used, so remove that one.

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


1 Files Affected:

- (modified) libc/src/__support/math/sqrtf128.h (-2) 


``````````diff
diff --git a/libc/src/__support/math/sqrtf128.h b/libc/src/__support/math/sqrtf128.h
index d5b54b724c259..14dcbec1acbb6 100644
--- a/libc/src/__support/math/sqrtf128.h
+++ b/libc/src/__support/math/sqrtf128.h
@@ -60,8 +60,6 @@ namespace math {
 
 namespace sqrtf128_internal {
 
-using FPBits = fputil::FPBits<float128>;
-
 template <typename T, typename U = T> LIBC_INLINE constexpr T prod_hi(T, U);
 
 // Get high part of integer multiplications.

``````````

</details>


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


More information about the libc-commits mailing list