[PATCH] D19553: Unroll pass restructure.
    Evgeny Stupachenko via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu May 26 01:52:40 PDT 2016
    
    
  
evstupac added a comment.
Thank you for the review and accept.
The patch is important for further improvements.
> could we make some named constant for the number of latch-associated instructions and use it instead of putting '2' everywhere?
That's a good point. I'll add a variable for this. However, generally it is not a constant value. It depends on loop instructions and unroll factor. We could have more recurrences to optimize. Say, "s ^= 1" will become constant for every unroll factor which is multiple of 2. I'm going to address this in one of further patches.
Repository:
  rL LLVM
http://reviews.llvm.org/D19553
    
    
More information about the llvm-commits
mailing list