[PATCH][x86] Add more patterns for SSE/AVX scalar	single/double-precision fp arithmetic instructions.
    Nadav Rotem 
    nrotem at apple.com
       
    Wed Dec 11 08:54:35 PST 2013
    
    
  
Andrea, 
The patterns look okay but I think that they may be fragile. Small changes in the IR passes or legalization may generate sequences that are not matched.  I mention this because it looks like this shuffle should be canonicalized into a select instruction (in InstCombine), because what it does is to blend two vectors: 
%2 = shufflevector <4 x float> %1, <4 x float> %a, <4 x i32> <i32 0, i32 5, i32 6, i32 7>
Can you please make sure that after canonicalizing the shuffle->blend your patterns still work?  
Thanks,
Nadav
On Dec 11, 2013, at 5:11 AM, Andrea Di Biagio <andrea.dibiagio at gmail.com> wrote:
> <patch.diff>
    
    
More information about the llvm-commits
mailing list