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

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 31 03:01:29 PDT 2018


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

I think this is fine to go in, a minor suggestion inline. Rest can be follow-up.



================
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())
----------------
Any reason you can't just do the peeling bit? Seems overly verbose to reset the rest.


Repository:
  rL LLVM

https://reviews.llvm.org/D53440





More information about the llvm-commits mailing list