[PATCH] D131486: [llvm][ADT] Allow using structured bindings with `llvm::enumerate`

[Jakub Kuderski via Phabricator via llvm-commits]

kuhar added inline comments.


================
Comment at: llvm/unittests/ADT/STLExtrasTest.cpp:94-95
 
   for (auto X : llvm::enumerate(foo)) {
     ++X.value();
   }
----------------
zero9178 wrote:
> kuhar wrote:
> > Can we also have a test that updates the value? I'm not sure if this is currently supported based on the `get` function.
> I did so already in line 81
Can we move that test into this testcase then?


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

https://reviews.llvm.org/D131486