[libcxx-commits] [libcxx] b05027a - Revert "[libc++][format] Use a helper constant."

Mark de Wever via libcxx-commits libcxx-commits at lists.llvm.org
Thu Apr 7 13:40:45 PDT 2022


Author: Mark de Wever
Date: 2022-04-07T22:40:08+02:00
New Revision: b05027aaf937714c2767a1fd93c3a5a76874609b

URL: https://github.com/llvm/llvm-project/commit/b05027aaf937714c2767a1fd93c3a5a76874609b
DIFF: https://github.com/llvm/llvm-project/commit/b05027aaf937714c2767a1fd93c3a5a76874609b.diff

LOG: Revert "[libc++][format] Use a helper constant."

This reverts commit 82427685ea3732665286d5b1c8a1424b1f96164e.

This seems to break the AIX-32 bit build.

Added: 
    

Modified: 
    libcxx/include/__format/formatter_pointer.h

Removed: 
    


################################################################################
diff  --git a/libcxx/include/__format/formatter_pointer.h b/libcxx/include/__format/formatter_pointer.h
index 539992fe866f1..aa4fc3398d096 100644
--- a/libcxx/include/__format/formatter_pointer.h
+++ b/libcxx/include/__format/formatter_pointer.h
@@ -46,12 +46,10 @@ class _LIBCPP_TEMPLATE_VIS __formatter_pointer : public __parser_pointer<_CharT>
     // but that code isn't public. Making that code public requires some
     // refactoring.
     // TODO FMT Remove code duplication.
-    constexpr size_t __max_hex_digits = 2 * sizeof(uintptr_t);
-    char __buffer[2 + __max_hex_digits];
+    char __buffer[2 + 2 * sizeof(uintptr_t)];
     __buffer[0] = '0';
     __buffer[1] = 'x';
-    char* __last =
-        __to_buffer(__buffer + 2, _VSTD::end(__buffer), reinterpret_cast<uintptr_t>(__ptr), __max_hex_digits);
+    char* __last = __to_buffer(__buffer + 2, _VSTD::end(__buffer), reinterpret_cast<uintptr_t>(__ptr), 16);
 
     unsigned __size = __last - __buffer;
     if (__size >= this->__width)


        


More information about the libcxx-commits mailing list