[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:12:24 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
----------------
Can you please restore this check line, possibly with an additional prefix?
================
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:
> 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.
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