From 0e373ecd52d595ebfc3054efdbacd7e6815bcd1e Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Fri, 19 Jul 2013 14:06:49 +0000 Subject: [PATCH] [SystemZ] Enable frame-pointer elimination for -O1 and above Copies the x86 behavior. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@186674 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Driver/Tools.cpp | 10 ++++++---- test/Driver/frame-pointer.c | 5 +++++ 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp index 87fbcd7688..8df0578e7d 100644 --- a/lib/Driver/Tools.cpp +++ b/lib/Driver/Tools.cpp @@ -1799,9 +1799,10 @@ static bool shouldUseFramePointer(const ArgList &Args, options::OPT_fomit_frame_pointer)) return A->getOption().matches(options::OPT_fno_omit_frame_pointer); - // Don't use a frame pointer on linux x86 and x86_64 if optimizing. + // Don't use a frame pointer on linux x86, x86_64 and z if optimizing. if ((Triple.getArch() == llvm::Triple::x86_64 || - Triple.getArch() == llvm::Triple::x86) && + Triple.getArch() == llvm::Triple::x86 || + Triple.getArch() == llvm::Triple::systemz) && Triple.getOS() == llvm::Triple::Linux) { if (Arg *A = Args.getLastArg(options::OPT_O_Group)) if (!A->getOption().matches(options::OPT_O0)) @@ -1817,9 +1818,10 @@ static bool shouldUseLeafFramePointer(const ArgList &Args, options::OPT_momit_leaf_frame_pointer)) return A->getOption().matches(options::OPT_mno_omit_leaf_frame_pointer); - // Don't use a leaf frame pointer on linux x86 and x86_64 if optimizing. + // Don't use a leaf frame pointer on linux x86, x86_64 and z if optimizing. if ((Triple.getArch() == llvm::Triple::x86_64 || - Triple.getArch() == llvm::Triple::x86) && + Triple.getArch() == llvm::Triple::x86 || + Triple.getArch() == llvm::Triple::systemz) && Triple.getOS() == llvm::Triple::Linux) { if (Arg *A = Args.getLastArg(options::OPT_O_Group)) if (!A->getOption().matches(options::OPT_O0)) diff --git a/test/Driver/frame-pointer.c b/test/Driver/frame-pointer.c index 6be395c598..027077ca5f 100644 --- a/test/Driver/frame-pointer.c +++ b/test/Driver/frame-pointer.c @@ -11,6 +11,11 @@ // RUN: %clang -target x86_64-pc-linux -### -S -O3 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK3-64 %s // RUN: %clang -target x86_64-pc-linux -### -S -Os %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECKs-64 %s +// Trust the above to get the optimizations right, and just test other targets +// that want this by default. +// RUN: %clang -target s390x-pc-linux -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-64 %s +// RUN: %clang -target s390x-pc-linux -### -S -O1 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK1-64 %s + // CHECK0-32: -mdisable-fp-elim // CHECK1-32-NOT: -mdisable-fp-elim // CHECK2-32-NOT: -mdisable-fp-elim -- 2.40.0