[PATCH] Optimize insertqi when we copy all the lower 64 bits.
    Nadav Rotem 
    nrotem at apple.com
       
    Tue Apr 15 11:04:07 PDT 2014
    
    
  
Hi Filipe, 
Why is this an IR-level transform? Could you implement this in SelectionDAG ?
Thanks,
Nadav
On Apr 15, 2014, at 10:48 AM, Jim Grosbach <grosbach at apple.com> wrote:
> +Nadav
> 
> Hi Filipe,
> 
> I like the idea of this transform. Nadav will have a better idea than I about whether this is the right place to go about it.
> 
> -Jim
> 
> On Apr 14, 2014, at 7:52 PM, Filipe Cabecinhas <filcab+llvm.phabricator at gmail.com> wrote:
> 
>> Updated the patch to add range merging, generating fewer insertqi when possible.
>> 
>> This also allows us to find more places to do the first opt.
>> 
>> Hi grosbach,
>> 
>> http://reviews.llvm.org/D3357
>> 
>> CHANGE SINCE LAST DIFF
>> http://reviews.llvm.org/D3357?vs=8482&id=8521#toc
>> 
>> Files:
>> lib/Transforms/InstCombine/InstCombineCalls.cpp
>> test/Transforms/InstCombine/vec_demanded_elts.ll
>> <D3357.2.patch>
> 
    
    
More information about the llvm-commits
mailing list