[PATCH] D145025: [ADT] Clean up `enumerate` implementation. NFC.

[Markus Böck via Phabricator via llvm-commits]

zero9178 accepted this revision.
zero9178 added a comment.
This revision is now accepted and ready to land.

LGTM % default constructor



================
Comment at: llvm/include/llvm/ADT/STLExtras.h:2200
 
-  result_pair() = default;
   result_pair(std::size_t Index, IterOfRange<R> Iter)
----------------
Any reason not having a default constructor is desired? I'd keep it just to avoid someone needing it in the future


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145025