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

[Konstantin Belousov]

On Fri, Nov 09, 2012 at 06:56:27PM +0000, 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 of upstream llvm
>   commit r126226, and it will be reverted by upstream too.
>   
>   MFC after:	1 week
> 
> Modified:
>   head/contrib/llvm/lib/Target/X86/X86Subtarget.cpp
> 
> Modified: head/contrib/llvm/lib/Target/X86/X86Subtarget.cpp
> ==============================================================================
> --- head/contrib/llvm/lib/Target/X86/X86Subtarget.cpp	Fri Nov  9 18:23:38 2012	(r242834)
> +++ head/contrib/llvm/lib/Target/X86/X86Subtarget.cpp	Fri Nov  9 18:56:27 2012	(r242835)
> @@ -416,12 +416,12 @@ X86Subtarget::X86Subtarget(const std::st
>    assert((!In64BitMode || HasX86_64) &&
>           "64-bit code requested on a subtarget that doesn't support it!");
>  
> -  // Stack alignment is 16 bytes on Darwin, FreeBSD, Linux and Solaris (both
> -  // 32 and 64 bit) and for all 64-bit targets.
> +  // Stack alignment is 16 bytes on Darwin, Linux and Solaris (both 32 and 64
> +  // bit) and for all 64-bit targets.
>    if (StackAlignOverride)
>      stackAlignment = StackAlignOverride;
> -  else if (isTargetDarwin() || isTargetFreeBSD() || isTargetLinux() ||
> -           isTargetSolaris() || In64BitMode)
> +  else if (isTargetDarwin() || isTargetLinux() || isTargetSolaris() ||
> +           In64BitMode)
>      stackAlignment = 16;
>  }
>  
Great, thank you for taking care of this.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20121109/7ee90d46/attachment.sig>