Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)
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/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. >> >> http://codereview.appspot.com/5488054/
Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)
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/tree-vect-stmts.c#newcode3712 >> >> tree-vect-stmts.c:3712: } >> The check can be put into a helper function. > > > Fixed. > > http://codereview.appspot.com/5488054/
Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)
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. http://codereview.appspot.com/5488054/
Re: [PATCH i386][google]With -mtune=core2, avoid generating the slow unaligned vector load/store (issue 5488054)
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/
