Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-12 Thread Bruce Evans
On Mon, 12 Nov 2012, Bruce Evans wrote: On Sun, 11 Nov 2012, Dimitry Andric wrote: It works just fine now with clang. For the first example, I get: pushl %ebp movl%esp, %ebp andl$-32, %esp as prolog, and for the second: pushl %ebp movl%es

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-11 Thread Konstantin Belousov
On Mon, Nov 12, 2012 at 02:01:47AM +1100, Bruce Evans wrote: > On Sun, 11 Nov 2012, Dimitry Andric wrote: > > > On 2012-11-11 12:53, Bruce Evans wrote: > >> > >> I'm not sure if either of us knows exactly what this does, but I like > >> this change. clang does stack alignment correctly, so it do

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-11 Thread Bruce Evans
On Sun, 11 Nov 2012, Dimitry Andric wrote: On 2012-11-11 12:53, Bruce Evans wrote: I'm not sure if either of us knows exactly what this does, but I like this change. clang does stack alignment correctly, so it doesn't need the gcc pessimization of aligning in the caller. clang aligns in the

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-11 Thread Dimitry Andric
On 2012-11-11 12:53, Bruce Evans wrote: On Sat, 10 Nov 2012, Nathan Whitehorn wrote: On 11/09/12 12:56, Dimitry Andric wrote: Log: Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as the FreeBSD ABI requires. This is essentially a revert of upstream llvm commit r1

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-11 Thread Dimitry Andric
On 2012-11-11 05:33, Nathan Whitehorn wrote: On 11/09/12 12:56, Dimitry Andric wrote: Author: dim Date: Fri Nov 9 18:56:27 2012 New Revision: 242835 URL: http://svnweb.freebsd.org/changeset/base/242835 Log: Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as the FreeB

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-11 Thread Bruce Evans
On Sat, 10 Nov 2012, Nathan Whitehorn wrote: On 11/09/12 12:56, Dimitry Andric wrote: Log: Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as the FreeBSD ABI requires. This is essentially a revert of upstream llvm commit r126226, and it will be reverted by upstream

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-10 Thread Nathan Whitehorn
On 11/09/12 12:56, Dimitry Andric wrote: Author: dim Date: Fri Nov 9 18:56:27 2012 New Revision: 242835 URL: http://svnweb.freebsd.org/changeset/base/242835 Log: Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as the FreeBSD ABI requires. This is essentially a revert o

Re: svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-09 Thread Konstantin Belousov
On Fri, Nov 09, 2012 at 06:56:27PM +, Dimitry Andric wrote: > Author: dim > Date: Fri Nov 9 18:56:27 2012 > New Revision: 242835 > URL: http://svnweb.freebsd.org/changeset/base/242835 > > Log: > Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as > the FreeBSD ABI requir

svn commit: r242835 - head/contrib/llvm/lib/Target/X86

2012-11-09 Thread Dimitry Andric
Author: dim Date: Fri Nov 9 18:56:27 2012 New Revision: 242835 URL: http://svnweb.freebsd.org/changeset/base/242835 Log: Reduce LLVM's default stack alignment for i386 from 16 to 4 bytes, as the FreeBSD ABI requires. This is essentially a revert of upstream llvm commit r126226, and it will