[PATCH] D70304: [Loop Peeling] Add possibility to enable peeling on loop nests.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Out of interest, do you have any performance & size numbers on the impact of this change?



================
Comment at: llvm/lib/Transforms/Utils/LoopUnrollPeel.cpp:292
 
   // Only try to peel innermost loops.
+  if (!UP.AllowLoopNestsPeeling && !L->empty())
----------------
Update comment.


================
Comment at: llvm/test/Transforms/LoopUnroll/peel-loop-conditions.ll:3
 ; RUN: opt < %s -S -loop-unroll -unroll-peel-max-count=4 -verify-dom-info | FileCheck %s
+; RUN: opt < %s -S -loop-unroll -unroll-peel-max-count=4 -unroll-allow-loop-nests-peeling -verify-dom-info | FileCheck %s --check-prefix LOOP-NEST
 
----------------
I think it would be better to add tests for this in a separate file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70304