[PATCH] D119342: [LoopRotate] Don't rotate loops when the minsize attribute is present

Ayke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 6 12:39:54 PST 2023


aykevl added inline comments.


================
Comment at: llvm/test/Transforms/LoopRotate/minsize-disable.ll:3
+; RUN: opt < %s -S -O2 -debug -debug-only=loop-rotate 2>&1 | FileCheck %s
+; RUN: opt < %s -S -passes='default<O2>' -debug -debug-only=loop-rotate 2>&1 | FileCheck %s
+
----------------
fhahn wrote:
> Does this test require running the full O2 pipeline? Usually we try to only run isolated passes if possible.
I basically just copied oz-disable.ll but yeah in this case only the looprotate pass is relevant. I have updated the patch accordingly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119342



More information about the llvm-commits mailing list