[PATCH] D18670: LoopUnroll: some small fixes/tweaks to make it more useful for partial unrolling

[Z. Zheng via llvm-commits]

zzheng added a comment.

In http://reviews.llvm.org/D18670#389991, @escha wrote:

> Ironically it looks like MaxCount itself doesn't even have a commandline option either... should I introduce one for both? :/ it feels wrong to bloat the commandline options like this, I guess




  static cl::opt<unsigned>                                                                                  
  UnrollCount("unroll-count", cl::Hidden,                                                                   
    cl::desc("Use this unroll count for all loops including those with "                                    
             "unroll_count pragma values, for testing purposes")); 

Is it reasonable to have one cmd option, similar to UnrollCount above, to curb both MaxCount and FullUnrollMaxCount?


Repository:
  rL LLVM

http://reviews.llvm.org/D18670