Skip to content
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

[libc][NFC] Fix missing replacement in BigInt #81864

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

gchatelet
Copy link
Contributor

No description provided.

@gchatelet gchatelet requested a review from lntue February 15, 2024 14:24
@llvmbot llvmbot added the libc label Feb 15, 2024
@llvmbot
Copy link
Collaborator

llvmbot commented Feb 15, 2024

@llvm/pr-subscribers-libc

Author: Guillaume Chatelet (gchatelet)

Changes

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

1 Files Affected:

  • (modified) libc/src/__support/UInt.h (+1-1)
diff --git a/libc/src/__support/UInt.h b/libc/src/__support/UInt.h
index 0828a34ba1a934..a1e9c4cda4e163 100644
--- a/libc/src/__support/UInt.h
+++ b/libc/src/__support/UInt.h
@@ -65,7 +65,7 @@ struct BigInt {
         val[i] = other[i];
     } else {
       size_t i = 0;
-      for (; i < OtherBits / 64; ++i)
+      for (; i < OtherBits / WORD_SIZE; ++i)
         val[i] = other[i];
       WordType sign = 0;
       if constexpr (Signed && OtherSigned) {

@gchatelet gchatelet merged commit 8416586 into llvm:main Feb 15, 2024
6 checks passed
@gchatelet gchatelet deleted the fix_missed_replacement branch February 15, 2024 14:31
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.

None yet

3 participants