[llvm-commits] [llvm] r40796 - /llvm/trunk/lib/Target/X86/X86TargetMachine.cpp

[Chris Lattner]

On Aug 3, 2007, at 3:46 PM, Dale Johannesen wrote:

> Author: johannes
> Date: Fri Aug  3 17:46:15 2007
> New Revision: 40796

Thanks Dale,

> ====================================================================== 
> ========
> --- llvm/trunk/lib/Target/X86/X86TargetMachine.cpp (original)
> +++ llvm/trunk/lib/Target/X86/X86TargetMachine.cpp Fri Aug  3  
> 17:46:15 2007
> @@ -116,7 +116,9 @@
>    : Subtarget(M, FS, is64Bit),
>      DataLayout(Subtarget.is64Bit() ?
>                 std::string("e-p:64:64-f64:32:64-i64:32:64- 
> f80:128:128") :
> -               std::string("e-p:32:32-f64:32:64-i64:32:64- 
> f80:128:128")),
> +                 Subtarget.isTargetDarwin() ?
> +                 std::string("e-p:32:32-f64:32:64-i64:32:64- 
> f80:128:128") :
> +                 std::string("e-p:32:32-f64:32:64-i64:32:64- 
> f80:32:32")),
>      FrameInfo(TargetFrameInfo::StackGrowsDown,
>                Subtarget.getStackAlignment(), Subtarget.is64Bit() ?  
> -8 : -4),
>      InstrInfo(*this), JITInfo(*this), TLInfo(*this) {

Dale, this is getting complicated :)

Can you please add a new X86Subtarget::getDataLayout() method?  That  
would make this just be:

      DataLayout(Subtarget.getDataLayout())

-Chris