[PATCH] D97290: [LoopInterchange] Replace tightly-nesting-ness check with the one from `LoopNest`

Ta-Wei Tu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 1 09:45:25 PST 2021


TaWeiTu added inline comments.


================
Comment at: llvm/include/llvm/Analysis/LoopNestAnalysis.h:161
+  /// have exactly one subloop and thus the loopnest is "totally-nested".
+  bool isTotallyNested() const {
+    return all_of(make_range(Loops.begin(), std::prev(Loops.end())),
----------------
Whitney wrote:
> is `totally-nested` a known term or you come up with it?
Ah, that's the term I came up with and that's why is quoted.
It will be nice if there's a more appropriate term for this.
Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97290



More information about the llvm-commits mailing list