Skip to content

[libc] Fix typo in uint128 type for PPC printf handling #138157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 1, 2025

Conversation

jhuber6
Copy link
Contributor

@jhuber6 jhuber6 commented May 1, 2025

No description provided.

@jhuber6 jhuber6 requested review from lntue and tstellar May 1, 2025 15:50
@llvmbot llvmbot added the libc label May 1, 2025
@llvmbot
Copy link
Member

llvmbot commented May 1, 2025

@llvm/pr-subscribers-libc

Author: Joseph Huber (jhuber6)

Changes

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

1 Files Affected:

  • (modified) libc/src/stdio/printf_core/core_structs.h (+1-1)
diff --git a/libc/src/stdio/printf_core/core_structs.h b/libc/src/stdio/printf_core/core_structs.h
index bab4fe75058a0..e27f77b6b594a 100644
--- a/libc/src/stdio/printf_core/core_structs.h
+++ b/libc/src/stdio/printf_core/core_structs.h
@@ -58,7 +58,7 @@ struct FormatSection {
   // Needs to be large enough to hold a long double. Special case handling for
   // the PowerPC double double type because it has no FPBits interface.
 #ifdef LIBC_TYPES_LONG_DOUBLE_IS_DOUBLE_DOUBLE
-  Uint128 conv_val_raw;
+  UInt128 conv_val_raw;
 #else
   fputil::FPBits<long double>::StorageType conv_val_raw;
 #endif // LIBC_TYPES_LONG_DOUBLE_IS_DOUBLE_DOUBLE

@jhuber6 jhuber6 merged commit 927cdd2 into llvm:main May 1, 2025
14 of 17 checks passed
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
GeorgeARM pushed a commit to GeorgeARM/llvm-project that referenced this pull request May 7, 2025
Ankur-0429 pushed a commit to Ankur-0429/llvm-project that referenced this pull request May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants