[patch] Simplify the datalayout strings by using the defaults

Richard Sandiford rsandifo at linux.vnet.ibm.com
Thu Dec 12 08:00:05 PST 2013


Rafael EspĂ­ndola <rafael.espindola at gmail.com> writes:
> diff --git a/lib/Target/SystemZ/SystemZTargetMachine.cpp b/lib/Target/SystemZ/SystemZTargetMachine.cpp
> index dee92e9..fd947ee 100644
> --- a/lib/Target/SystemZ/SystemZTargetMachine.cpp
> +++ b/lib/Target/SystemZ/SystemZTargetMachine.cpp
> @@ -30,8 +30,7 @@ SystemZTargetMachine::SystemZTargetMachine(const Target &T, StringRef TT,
>      // Make sure that global data has at least 16 bits of alignment by default,
>      // so that we can refer to it using LARL.  We don't have any special
>      // requirements for stack variables though.
> -    DL("E-p:64:64:64-i1:8:16-i8:8:16-i16:16-i32:32-i64:64"
> -       "-f32:32-f64:64-f128:64-a0:8:16-n32:64"),
> +    DL("E-p:64:64:64-i1:8:16-i8:8:16-i64:64-f128:64-a:8:16-n32:64"),
>      InstrInfo(*this), TLInfo(*this), TSInfo(*this),
>      FrameLowering(*this, Subtarget) {
>    initAsmInfo();

Looks good to me, thanks.  If we don't want "a0:" to be used then
maybe it would be worth changing:

-  ``a0:0:64`` - aggregates are 64-bit aligned

in LangRef.rst too.

Richard





More information about the llvm-commits mailing list