[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