[PATCH] D62938: [SLP] Forbid to vectorize bundles with same opcode but different IR flags
    Simon Pilgrim via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jun  6 14:19:28 PDT 2019
    
    
  
RKSimon added a comment.
In D62938#1532286 <https://reviews.llvm.org/D62938#1532286>, @lebedev.ri wrote:
> Please can you add a better explanation of the problem to the description of the patch?
>  I'm not sure what the problem is, you are allowed to drop `nuw`/`nsw` flags: https://rise4fun.com/Alive/plNm
>  So the new vectorized binop should simply take the smallest common subset of flags, which likely most often means no flags.
@dtemirbulatov It might be worth adding tests that check that we do retain the common subset?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62938/new/
https://reviews.llvm.org/D62938
    
    
More information about the llvm-commits
mailing list