Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

Twine now stores integers by value, so this patch updates comments
accordingly.

Twine now stores integers by value, so this patch updates comments
accordingly.
@llvmbot
Copy link
Member

llvmbot commented Sep 16, 2025

@llvm/pr-subscribers-llvm-adt

Author: Kazu Hirata (kazutakahirata)

Changes

Twine now stores integers by value, so this patch updates comments
accordingly.


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

1 Files Affected:

  • (modified) llvm/include/llvm/ADT/Twine.h (+5-8)
diff --git a/llvm/include/llvm/ADT/Twine.h b/llvm/include/llvm/ADT/Twine.h
index 249fb0ad83808..d9f9c0f0d5d9c 100644
--- a/llvm/include/llvm/ADT/Twine.h
+++ b/llvm/include/llvm/ADT/Twine.h
@@ -119,22 +119,19 @@ class Twine {
     /// An int value, to render as a signed decimal integer.
     DecIKind,
 
-    /// A pointer to an unsigned long value, to render as an unsigned decimal
-    /// integer.
+    /// An unsigned long value, to render as an unsigned decimal integer.
     DecULKind,
 
-    /// A pointer to a long value, to render as a signed decimal integer.
+    /// A long value, to render as a signed decimal integer.
     DecLKind,
 
-    /// A pointer to an unsigned long long value, to render as an unsigned
-    /// decimal integer.
+    /// An unsigned long long value, to render as an unsigned decimal integer.
     DecULLKind,
 
-    /// A pointer to a long long value, to render as a signed decimal integer.
+    /// A long long value, to render as a signed decimal integer.
     DecLLKind,
 
-    /// A pointer to a uint64_t value, to render as an unsigned hexadecimal
-    /// integer.
+    /// A uint64_t value, to render as an unsigned hexadecimal integer.
     UHexKind
   };
 

@kazutakahirata kazutakahirata merged commit 3938ce9 into llvm:main Sep 16, 2025
11 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_20250915_ADT_Twine branch September 16, 2025 16:07
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.

4 participants