Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-14 Thread Sriraman Tallam
On Wed, Dec 14, 2011 at 9:47 AM, Xinliang David Li wrote: > The patch is ok for google branches for now. Committed to google/main. Thanks, -Sri. > > David > > On Tue, Dec 13, 2011 at 4:33 PM,   wrote: >> On 2011/12/14 00:04:10, davidxl wrote: >>> >>> http://codereview.appspot.com/5488054/diff/4

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-14 Thread Xinliang David Li
The patch is ok for google branches for now. David On Tue, Dec 13, 2011 at 4:33 PM, wrote: > On 2011/12/14 00:04:10, davidxl wrote: >> >> http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c >> File tree-vect-stmts.c (right): > > > > http://codereview.appspot.com/5488054/diff/4002/

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-13 Thread tmsriram
On 2011/12/14 00:04:10, davidxl wrote: http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c File tree-vect-stmts.c (right): http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c#newcode3712 tree-vect-stmts.c:3712: } The check can be put into a helper function. Fixed

Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)

2011-12-13 Thread davidxl
http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c File tree-vect-stmts.c (right): http://codereview.appspot.com/5488054/diff/4002/tree-vect-stmts.c#newcode3712 tree-vect-stmts.c:3712: } The check can be put into a helper function. http://codereview.appspot.com/5488054/