It seems that one of the tests needed a small fix. Attached is a corrected version.
On 20 June 2013 17:16, Michael Zolotukhin <michael.v.zolotuk...@gmail.com> wrote: > Hi, > I added two tests to verify we generate vector instructions when > vector_loop is used. > Is the patch ok with that change? > > Thanks, Michael > > On 5 June 2013 18:10, Michael Zolotukhin <michael.v.zolotuk...@gmail.com> > wrote: >> I'll prepare some tests shortly, >> What about the rest questions? >> >> Thanks, Michael >> >> On 15 May 2013 19:45, H.J. Lu <hjl.to...@gmail.com> wrote: >>> On Wed, May 15, 2013 at 5:47 AM, Michael Zolotukhin >>> <michael.v.zolotuk...@gmail.com> wrote: >>>> Hi HJ, >>>>> You use Pmode as the largest integer mode. Is word_mode better >>>>> than Pmode since word_mode is DI and Pmode may be SI for x32. >>>> I updated the patch to use word_mode instead of Pmode. Bootstrap, >>>> make check and Specs2000 are ok. >>>> >>>> Thanks, Michael >>>> >>>> On 14 May 2013 19:55, H.J. Lu <hjl.to...@gmail.com> wrote: >>>>> On Tue, May 14, 2013 at 7:34 AM, Michael Zolotukhin >>>>> <michael.v.zolotuk...@gmail.com> wrote: >>>>>> Hi, >>>>>> I attached an updated version of the patch. Looks like the 64-bit issue >>>>>> is >>>>>> resolved in it (now a vector mode is explicitly chosen instead of TI- or >>>>>> another integer mode). Also, some of remarks are fixed in it - some >>>>>> others are still not changed, because I don't know what to do with them >>>>>> right >>>>>> now (see my questions in the previous letter). >>>>>> >>>>>> Could you take a look at the patch? >>>>>> >>>>>> I checked it on i386/x86_64 bootstrap and make check, and on >>>>>> Specs2000/2006 >>>>>> (I checked only stability). >>>>>> >>>>> >>>>> You use Pmode as the largest integer mode. Is word_mode better >>>>> than Pmode since word_mode is DI and Pmode may be SI for x32. >>>>> >>>>> >>> >>> You should include tests to verify that it works as >>> expected. >>> >>> >>> -- >>> H.J. >> >> >> >> -- >> --- >> Best regards, >> Michael V. Zolotukhin, >> Software Engineer >> Intel Corporation. > > > > -- > --- > Best regards, > Michael V. Zolotukhin, > Software Engineer > Intel Corporation. -- --- Best regards, Michael V. Zolotukhin, Software Engineer Intel Corporation.
memmov-5.patch
Description: Binary data
