[PATCH] D82895: [NFC][LoopInfo] Document empty()

Ettore Tiotto via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 14:36:20 PDT 2020


etiotto added a comment.

In D82895#2129028 <https://reviews.llvm.org/D82895#2129028>, @jdoerfert wrote:

> > I think that isInnermost() would make even more sense.
>
> I'm fine with adding `bool isInnermost() const { return empty(); }` and while we are at it, `bool isOutermost()`.
>
> @fhahn @Meinersbur @Whitney @etiotto Objections?


I think the proposed name `isInnermost` is fine. No objections on my part.


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

https://reviews.llvm.org/D82895





More information about the llvm-commits mailing list