search for: expandinserttovectorthroughstack

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! > > >...

Alright, go ahead with it. —Owen > On Nov 17, 2014, at 4:58 AM, Daniil Troshkov <troshkovdanil at gmail.com> wrote: > > 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 <mailto:troshkovdanil at gmail.com>> wrote...

...patch is OK, please, commit it . Danil. -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20141112/52592eeb/attachment.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: ExpandInsertToVectorThroughStack.patch Type: application/octet-stream Size: 2960 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20141112/52592eeb/attachment.obj>