[PATCH] D144503: [ADT] Allow `llvm::enumerate` to enumerate over multiple ranges

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 16:10:04 PDT 2023


kuhar added a comment.

In D144503#4203254 <https://reviews.llvm.org/D144503#4203254>, @aprantl wrote:

> This somehow broke the `-DLLVM_ENABLE_MODULES=1` build:
>
> Can you take a look?
>
> https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/52513/consoleFull#-458239162a1ca8a51-895e-46c6-af87-ce24fa4cd561

Hi @aprantl,

What are the requirements for building with modules enabled? I tried this locally but it didn't get very far, not sure if my system compiler & standard library are new enough (clang14).
If you can reproduce it on your end, could you check if applying D146231 <https://reviews.llvm.org/D146231> fixes it? My guess is that we may need to 'normalize' types passed to all_equal to `size_t`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144503/new/

https://reviews.llvm.org/D144503



More information about the llvm-commits mailing list