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

Steven Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 09:43:53 PDT 2023


srj added a comment.

In D144503#4201050 <https://reviews.llvm.org/D144503#4201050>, @srj wrote:

> In D144503#4201046 <https://reviews.llvm.org/D144503#4201046>, @srj wrote:
>
>> Build environment is fairly stock Visual Studio Community 2022 on Windows 10.  (I'll go check to see if there are updates that need installing, but they should be pretty close to the latest.)
>
> EDIT: looks like there are some uninstalled updates -- I'll get those installed and re-check, maybe this is just an MSVC bug.

So after installing the latest VS2022 updates, things appear to be working smoothly!

I was not expecting this at all -- it's been a long time since I've seen a miscompilation of this sort in any modern compiler so I didn't even think to check this.

Sorry for the noise!


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