[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