[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