[all-commits] [llvm/llvm-project] eea5e7: [libc++][string] Add regression test for sized new...

Vitaly Buka via All-commits all-commits at lists.llvm.org
Mon Sep 30 22:29:32 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: eea5e7e095f945329e9e71c9606075264df8d74f
      https://github.com/llvm/llvm-project/commit/eea5e7e095f945329e9e71c9606075264df8d74f
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2024-09-30 (Mon, 30 Sep 2024)

  Changed paths:
    A libcxx/test/std/strings/basic.string/string.capacity/deallocate_size.pass.cpp

  Log Message:
  -----------
  [libc++][string] Add regression test for sized new/delete bug (#110210)

This is regression test for #90292.

Allocator used in test is very similar to test_allocator.
However, reproducer requires size_type of the string
to be 64bit, but test_allocator uses 32bit.

32bit size_type makes `sizeof(string::__long)` to be 16,
but the alignment issue fixed with #90292 is only triggered
with default `sizeof(string::__long)` which is 24.

Fixes #92128.

---------

Co-authored-by: Louis Dionne <ldionne.2 at gmail.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list