[libcxx-commits] [libcxx] [libc++] Fix unintended ABI break in associative containers with reference comparators (PR #118685)

via libcxx-commits libcxx-commits at lists.llvm.org
Wed Dec 4 11:13:16 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 842d56315108a10a0b5f866d7a44dc676f5d2586 e7d3f31effdf5b49f4fe69e9d005f6ea4c7c2f50 --extensions cpp,h -- libcxx/test/libcxx/containers/associative/reference_comparator_abi.compile.pass.cpp libcxx/include/__memory/compressed_pair.h
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/libcxx/include/__memory/compressed_pair.h b/libcxx/include/__memory/compressed_pair.h
index 7687615bd0..4a45f6d2ea 100644
--- a/libcxx/include/__memory/compressed_pair.h
+++ b/libcxx/include/__memory/compressed_pair.h
@@ -52,7 +52,8 @@ _LIBCPP_BEGIN_NAMESPACE_STD
 // member3 - offset: 8, size: 8
 //
 // Furthermore, that alignment must be the same as what was used in the old __compressed_pair layout, so we must
-// handle reference types specially since alignof(T&) == alignof(T). See https://github.com/llvm/llvm-project/issues/118559.
+// handle reference types specially since alignof(T&) == alignof(T). See
+// https://github.com/llvm/llvm-project/issues/118559.
 
 #ifndef _LIBCPP_ABI_NO_COMPRESSED_PAIR_PADDING
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/118685


More information about the libcxx-commits mailing list