[PATCH] D109397: [AttributeList] Change indexes in AttributeList::AttrIndex

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

the main reason I wanted to do this was to avoid the weird unsigned wrapping when doing

  for (unsigned i = AL.index_begin(), e = AL.index_end(); i != e; ++i)

but maybe that's not a good enough reason

perhaps instead we could remove the `index_begin`/`index_end` methods and provide an iterator via something like `indexes`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109397