[PATCH] D148071: [LoopUnroll] Prevent LoopFullUnrollPass to perform partial/runtime unrolling

[Yashwant Singh via Phabricator via llvm-commits]

yassingh marked an inline comment as done.
yassingh added inline comments.


================
Comment at: llvm/test/Transforms/LoopUnroll/revisit.ll:11
 ; Also run in a special mode that visits children.
 ; RUN: opt < %s -disable-output -debug-pass-manager -unroll-revisit-child-loops 2>&1 \
 ; RUN: -passes='require<opt-remark-emit>,loop(loop-unroll-full)' \
----------------
nikic wrote:
> yassingh wrote:
> > How can I add an appropriate test for this option "unroll-revisit-child-loops"? 
> I think you can just drop the option entirely, it no longer makes sense. We can probably `assert(!IsCurrentLoopValid)` now: Full unrolling should always invalidate the current loop.
Removed this run line. I  can submit another change to entirely drop this cl option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148071