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

[Steven Johnson via Phabricator via llvm-commits]

srj added a comment.

According to `git bisect`, this change appears to have broken `llvm-tblgen` when building under MSVC

  C:\build_bot\worker\llvm-17-x86-64-windows\llvm-project\llvm\utils\TableGen\IntrinsicEmitter.cpp(845): error C2440: 'initializing':
  cannot convert from 'size_t' to '_This'
          with
          [
              _This=const llvm::SmallVector<llvm::CodeGenIntrinsic::ArgAttribute,0> &
          ]
  C:\build_bot\worker\llvm-17-x86-64-windows\llvm-project\llvm\utils\TableGen\IntrinsicEmitter.cpp(845): note: Reason: cannot convert
  from 'size_t' to 'const T'
          with
          [
              T=llvm::SmallVector<llvm::CodeGenIntrinsic::ArgAttribute,0>
          ]
  C:\build_bot\worker\llvm-17-x86-64-windows\llvm-project\llvm\utils\TableGen\IntrinsicEmitter.cpp(845): note: Constructor for class '
  llvm::SmallVector<llvm::CodeGenIntrinsic::ArgAttribute,0>' is declared 'explicit'

this has rendered all Windows-based testing of Halide impossible; I urge someone to fix (or revert) this change as soon as conveniently possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144503