[PATCH] D148071: [LoopUnroll] Prevent LoopFullUnrollPass to perform partial/runtime unrolling
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 12 06:41:45 PDT 2023
nikic added inline comments.
================
Comment at: llvm/test/Transforms/LoopUnroll/full-unroll-keep-first-exit.ll:3
; RUN: opt -S -passes=loop-unroll < %s | FileCheck %s
-; RUN: opt -S -passes='require<opt-remark-emit>,loop(loop-unroll-full)' < %s | FileCheck %s
----------------
nikic wrote:
> Can you please restore this check line, possibly with an additional prefix?
This should use `--check-prefixes=CHECK,CHECK-FULL` to deduplicate the cases that are the same between full/non-full.
================
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)' \
----------------
yassingh wrote:
> 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.
That's fine.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148071/new/
https://reviews.llvm.org/D148071
More information about the llvm-commits
mailing list