Skip to content

Commit b9b6a63

Browse files
committed
Update clang comments.
1 parent 3b71196 commit b9b6a63

3 files changed

Lines changed: 1 addition & 8 deletions

File tree

include/bitcoin/system/have.hpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -204,9 +204,6 @@
204204
#endif
205205
#endif
206206

207-
/// C++20: parenthesized initialization of aggregates requires clang16/xcode16.
208-
/// We don't have macro treatment for that, just ad-hoc HAVE_CLANG conditions.
209-
210207
/// None on xcode.
211208
/// Requires link with -ltbb on gcc (v9).
212209
/// Experimental on clang (libcxx.llvm.org/Status/PSTL.html), requires:

include/bitcoin/system/literals.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ namespace libbitcoin {
3535
/// not be subject to regressions in other code, as a break here causes a large
3636
/// number of test and other failures. See tests for usage and detailed info.
3737
/// ---------------------------------------------------------------------------
38-
/// These should be consteval for safety, but waiting on clang++20 to catch up.
38+
/// These should be consteval for safety, but waiting on clang++ v17.
3939

4040
// Functions are consteval where available (bogus warning).
4141
BC_PUSH_WARNING(USE_CONSTEXPR_FOR_FUNCTION)

test/allocator.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,6 @@ BC_PUSH_WARNING(NO_UNGUARDED_POINTERS)
2626
BC_PUSH_WARNING(NO_DELETE_RAW_POINTER)
2727
BC_PUSH_WARNING(NO_UNCLEARED_OWNER_POINTER)
2828

29-
// Clang is not yet C++20 compliant in terms of aggregate initialization.
30-
// See [reviews.llvm.org/D140327] for details. So vectors provide constructors.
31-
///////////////////////////////////////////////////////////////////////////////
32-
3329
struct simple
3430
{
3531
simple(size_t arg={}) NOEXCEPT

0 commit comments

Comments
 (0)