[PATCH] D159126: [Clang] Add captures to the instantiation scope of lambda call operators
Alex Brachet via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 6 17:11:19 PDT 2023
abrachet added a comment.
I'm seeing libcxx failures after this patch. See https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-linux-x64/b8770673168839783889/overview
# .---command stderr------------
# | /b/s/w/ir/x/w/llvm-llvm-project/libcxx/test/std/ranges/range.adaptors/range.all/range.owning.view/empty.pass.cpp:38:19: error: static assertion failed due to requirement '!HasEmpty<const std::ranges::owning_view<ComparableIters> &>'
# | 38 | static_assert(!HasEmpty<const OwningView&>);
# | | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
# | /b/s/w/ir/x/w/llvm-llvm-project/libcxx/test/std/ranges/range.adaptors/range.all/range.owning.view/empty.pass.cpp:39:19: error: static assertion failed due to requirement '!HasEmpty<const std::ranges::owning_view<ComparableIters> &&>'
# | 39 | static_assert(!HasEmpty<const OwningView&&>);
# | | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# | 2 errors generated.
# `-----------------------------
I've verified that reverting this patch fixes those errors. Would you mind taking a look?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159126/new/
https://reviews.llvm.org/D159126
More information about the cfe-commits
mailing list