[LLVMdev] [llvm][SelectionDAG] trivial patch: fix misprint in SelectionDAGLegalize::ExpandInsertToVectorThroughStack

[Daniil Troshkov]

Hi!

I have not found test case. (It is because we have no target using
"ExpandInsertToVectorThroughStack").
But I tested it for target currently not included in llvm trunk.

This fix correct and trivial, so I'm offering to commit it...

On Wed, Nov 12, 2014 at 11:47 PM, Daniil Troshkov <troshkovdanil at gmail.com>
wrote:

> I detected this bug using test case from platform which is not currently
> supported on llvm targets. (Our team is porting llvm on new target).
> Creating the test case will take some extra time. I'll try to do it ASAP.
> Have you any ideas about the test case? (targets using
> ExpandInsertToVectorThroughStack, etc...)
>
>
> On Wed, Nov 12, 2014 at 8:29 PM, Owen Anderson <resistor at mac.com> wrote:
>
>> LGTM.  Can you supply a testcase?
>>
>> —Owen
>>
>> > On Nov 12, 2014, at 5:52 AM, Daniil Troshkov <troshkovdanil at gmail.com>
>> wrote:
>> >
>> > Hi Owen!
>> >
>> > The "First store the whole vector" is without uses and will be deleted
>> later.
>> > I've attached trivial patch to fix it.
>> > I have no commit access so if patch is OK, please, commit it .
>> >
>> > Danil.
>> > <ExpandInsertToVectorThroughStack.patch>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20141117/8548c689/attachment.html>