[PATCH] D78796: [Support] Refactor LEB128 encoding into an input iterator

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 15 16:35:53 PDT 2020


dblaikie added a comment.

Sorry for the delay coming back to this.

If there's sufficient agreement/justification/push to have an iterator interface, I think it'd be OK/maybe better to just have that, rather than the two - in either wrapping order. I do appreciate the "range-like" object (ULEBifier or whatever it's called) to make it easier to use with range-based algorithms, etc.


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

https://reviews.llvm.org/D78796





More information about the llvm-commits mailing list