[PATCH] D57618: [ADT] Add a fallible_iterator wrapper.

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 14:42:46 PST 2019


alexshap added inline comments.


================
Comment at: include/llvm/ADT/fallible_iterator.h:53
+///     Error Err = Error::success();
+///     for (auto &C : A.children()) {
+///       // Loop body only entered when increment succeeds.
----------------
looks like the argument (Err) is missed here


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57618





More information about the llvm-commits mailing list