I attached the test and patch that were already attached to the bug. The text in the comment associated with patch attachment is:
The comment above the code I changed is:
// For example,
// I4I4LDID 41 <1,4,.preg_I4> T<4,.predef_I4,4> # i
// U4INTCONST 8 (0x8)
// I4MPY
// U8I4CVT
// U8U8LDID 0 <2,1,rowsptr2> T<32,anon_ptr.,8>
// U8ADD
// This can be safely converted into (without performance loss):
// I8INTCONST 8 (0x8)
// I8I4LDID 41 <1,4,.preg_I4> T<4,.predef_I4,4> # i
// I8MPY
// U8U8LDID 0 <2,1,rowsptr2> T<32,anon_ptr.,8>
// U8ADD
// We will do this only for small powers of 2 so that
// the I8MPY gets converted into shifts (no imul64 versus imul32).
The original code does not implement this transformation,
the supplied patch I believe does.
Could a gatekeeper review/approve this change when he or she has a chance?
Thanks,
Doug
/* { dg-do compile } */
/* { dg-options "-O3" } */
/* extracted from ffmpeg libavcodec/dsputil.c */
void put_pixels2_l2(unsigned char *dst);
void
put_h264_qpel2_mc01_c(unsigned char *dst, unsigned char *src, int stride)
{
int i;
unsigned char *csrc = src - stride * 2;
for (i = 0; i < 7; i++)
{
*((unsigned short *) dst) = *((unsigned short *) csrc);
dst += 2;
csrc += stride;
}
put_pixels2_l2(dst);
}
bug755.patch
Description: bug755.patch
------------------------------------------------------------------------------ Create and publish websites with WebMatrix Use the most popular FREE web apps or write code yourself; WebMatrix provides all the features you need to develop and publish your website. http://p.sf.net/sfu/ms-webmatrix-sf
_______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
