[PATCH] D53440: [LoopUnroll] allow customization for new-pass-manager version of LoopUnroll

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added inline comments.


================
Comment at: lib/Passes/PassRegistry.def:218
 FUNCTION_PASS("unroll", LoopUnrollPass())
+FUNCTION_PASS("unroll<peeling>",LoopUnrollPass(LoopUnrollOptions().setPartial(false).setPeeling(true).setRuntime(false).setUpperBound(false)))
 FUNCTION_PASS("verify", VerifierPass())
----------------
chandlerc wrote:
> Any reason you can't just do the peeling bit? Seems overly verbose to reset the rest.
Because it needs to be a restricted set, not allowing generic unroll.
At least it needs no-runtime. Will try to keep it to the minimal set.


Repository:
  rL LLVM

https://reviews.llvm.org/D53440