[libcxx-commits] [PATCH] D145520: [libc++][test] Uses qualified std::uint32_t.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Mar 7 11:29:05 PST 2023


Mordante created this revision.
Mordante added a reviewer: ldionne.
Herald added a subscriber: miyuki.
Herald added a project: All.
Mordante requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

The module std does not provide c-types in the global namespace. This
means all these types need to be fully qualified. This is a first step
to convert them by using sed.

Since this is an automated conversion other types like uint64_t are kept
as is.

Note that tests in the directory libcxx/test/std/depr/depr.c.headers
should not be converted automatically. This requires manual attention,
there some test require testing uint32_t in the global namespace. These
test should fail when using the std module, and pass when using the
std.compat module.

A similar issue occurs with atomic, atomic_uint32_t is specified as

  using atomic_uint32_t = atomic<uint32_t>; // freestanding

So here too we need to keep the name in the global namespace in the
tests.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D145520

Files:
  libcxx/test/libcxx/numerics/numeric.ops/midpoint.integer.pass.cpp
  libcxx/test/libcxx/type_traits/convert_to_integral.pass.cpp
  libcxx/test/libcxx/type_traits/is_always_bitcastable.compile.pass.cpp
  libcxx/test/libcxx/utilities/function.objects/unord.hash/murmur2_or_cityhash_ubsan_unsigned_overflow_ignored.pass.cpp
  libcxx/test/std/atomics/atomics.types.generic/integral.pass.cpp
  libcxx/test/std/atomics/types.pass.cpp
  libcxx/test/std/containers/sequences/vector/vector.cons/construct_iter_iter.pass.cpp
  libcxx/test/std/experimental/simd/simd.cons/broadcast.pass.cpp
  libcxx/test/std/experimental/simd/simd.cons/load.pass.cpp
  libcxx/test/std/experimental/simd/simd.mem/load.pass.cpp
  libcxx/test/std/experimental/simd/simd.traits/is_abi_tag.pass.cpp
  libcxx/test/std/experimental/simd/simd.traits/is_simd.pass.cpp
  libcxx/test/std/experimental/simd/simd.traits/is_simd_mask.pass.cpp
  libcxx/test/std/input.output/iostream.format/output.streams/ostream.formatted/ostream.inserters.arithmetic/minus1.pass.cpp
  libcxx/test/std/numerics/bit/bit.endian/endian.pass.cpp
  libcxx/test/std/numerics/bit/bit.pow.two/bit_ceil.fail.cpp
  libcxx/test/std/numerics/bit/bit.pow.two/bit_ceil.pass.cpp
  libcxx/test/std/numerics/bit/bit.pow.two/bit_floor.pass.cpp
  libcxx/test/std/numerics/bit/bit.pow.two/bit_width.pass.cpp
  libcxx/test/std/numerics/bit/bit.pow.two/has_single_bit.pass.cpp
  libcxx/test/std/numerics/bit/bitops.count/countl_one.pass.cpp
  libcxx/test/std/numerics/bit/bitops.count/countl_zero.pass.cpp
  libcxx/test/std/numerics/bit/bitops.count/countr_one.pass.cpp
  libcxx/test/std/numerics/bit/bitops.count/countr_zero.pass.cpp
  libcxx/test/std/numerics/bit/bitops.count/popcount.pass.cpp
  libcxx/test/std/numerics/bit/bitops.rot/rotl.pass.cpp
  libcxx/test/std/numerics/bit/bitops.rot/rotr.pass.cpp
  libcxx/test/std/numerics/bit/byteswap.pass.cpp
  libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.integer.pass.cpp
  libcxx/test/std/utilities/charconv/charconv.msvc/floating_point_test_cases.hpp
  libcxx/test/std/utilities/charconv/charconv.msvc/test.cpp
  libcxx/test/std/utilities/format/format.functions/escaped_output.ascii.pass.cpp
  libcxx/test/std/utilities/format/format.functions/escaped_output.unicode.pass.cpp
  libcxx/test/std/utilities/format/format.functions/format_tests.h
  libcxx/test/std/utilities/function.objects/unord.hash/integral.pass.cpp
  libcxx/test/support/atomic_helpers.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145520.503111.patch
Type: text/x-patch
Size: 43392 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230307/039eda31/attachment-0001.bin>


More information about the libcxx-commits mailing list