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

Mon Mar 20 10:30:49 PDT 2023

kuhar added a comment.

In D144503#4206725 <https://reviews.llvm.org/D144503#4206725>, @dstuttard wrote:

> In D144503#4204120 <https://reviews.llvm.org/D144503#4204120>, @ronlieb wrote:
>
>> also seeing similar error in downstream windows build of llvm.
>>
>> In D144503#4201034 <https://reviews.llvm.org/D144503#4201034>, @srj wrote:
>>
>>> There is also a similar breakage in InstrInfoEmitter.cpp:
>>>
>>>   C:\build_bot\worker\llvm-17-x86-32-windows\llvm-project\llvm\utils\TableGen\InstrInfoEmitter.cpp(120): error C2440: 'initializing': cannot convert from 'size_t' to '_This'
>>>           with
>>>           [
>>>               _This=llvm::Record *const &
>>>           ]
>
> I note from an earlier comment that updating VS2022 with latest patches solved this issue for @srj 
> But is the issue similar for the minimum required version of MSVC (currently Visual Studio 2019 / v16.0 )??

It passed across all windows buildbots, which includes VS 2019. Example: https://lab.llvm.org/buildbot/#/builders/13/builds/33254

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144503