[PATCH] D59710: [SLP] remove lower limit for forming reduction patterns
    Simon Pilgrim via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Mar 22 14:35:52 PDT 2019
    
    
  
RKSimon added a comment.
In D59710#1440002 <https://reviews.llvm.org/D59710#1440002>, @ABataev wrote:
> It really requires some additional improvvements, because I see a lot of regressions for the cmp instructions. I think, at first you should try to vectorize cmp instructions using the horizontal reductions anf only if it was unsuccessful, you need to try to vectorize the operands of the instruction itself.
@abataev Please can you be more specific on what icmp issues you mean? All I'm mainly seeing is a lot of scalar leftovers that InstCombine/InstSimplify will clean up if we added them as a pass in the tests.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59710/new/
https://reviews.llvm.org/D59710
    
    
More information about the llvm-commits
mailing list