r296861 - [PowerPC] Enable -fomit-frame-pointer by default for PPC

[Hans Wennborg via cfe-commits]

Should we add a mention of this to the release notes?

On Fri, Mar 3, 2017 at 1:49 AM, Nemanja Ivanovic via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: nemanjai
> Date: Fri Mar  3 03:49:17 2017
> New Revision: 296861
>
> URL: http://llvm.org/viewvc/llvm-project?rev=296861&view=rev
> Log:
> [PowerPC] Enable -fomit-frame-pointer by default for PPC
>
> As is the case on platforms like Mips, X86 and SystemZ, the -fomit-frame-pointer
> should be enabled by default on PPC when optimizing at -O1 and above. This
> brings the behaviour of LLVM on PPC in line with GCC.
>
> Committing on behalf of Hiroshi Inoue.
>
> Differential Revision: https://reviews.llvm.org/D29750
>
> Modified:
>     cfe/trunk/lib/Driver/Tools.cpp
>     cfe/trunk/test/Driver/frame-pointer.c
>
> Modified: cfe/trunk/lib/Driver/Tools.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=296861&r1=296860&r2=296861&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/Tools.cpp (original)
> +++ cfe/trunk/lib/Driver/Tools.cpp Fri Mar  3 03:49:17 2017
> @@ -2235,6 +2235,9 @@ static bool useFramePointerForTargetByDe
>      case llvm::Triple::mips64el:
>      case llvm::Triple::mips:
>      case llvm::Triple::mipsel:
> +    case llvm::Triple::ppc:
> +    case llvm::Triple::ppc64:
> +    case llvm::Triple::ppc64le:
>      case llvm::Triple::systemz:
>      case llvm::Triple::x86:
>      case llvm::Triple::x86_64:
>
> Modified: cfe/trunk/test/Driver/frame-pointer.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/frame-pointer.c?rev=296861&r1=296860&r2=296861&view=diff
> ==============================================================================
> --- cfe/trunk/test/Driver/frame-pointer.c (original)
> +++ cfe/trunk/test/Driver/frame-pointer.c Fri Mar  3 03:49:17 2017
> @@ -17,6 +17,13 @@
>  // 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
>
> +// RUN: %clang -target powerpc-unknown-linux-gnu -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-64 %s
> +// RUN: %clang -target powerpc-unknown-linux-gnu -### -S -O1 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK1-64 %s
> +// RUN: %clang -target powerpc64-unknown-linux-gnu -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-64 %s
> +// RUN: %clang -target powerpc64-unknown-linux-gnu -### -S -O1 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK1-64 %s
> +// RUN: %clang -target powerpc64le-unknown-linux-gnu -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-64 %s
> +// RUN: %clang -target powerpc64le-unknown-linux-gnu -### -S -O1 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK1-64 %s
> +
>  // RUN: %clang -target mips-linux-gnu -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-32 %s
>  // RUN: %clang -target mips-linux-gnu -### -S -O1 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK1-32 %s
>  // RUN: %clang -target mipsel-linux-gnu -### -S -O0 %s -o %t.s 2>&1 | FileCheck -check-prefix=CHECK0-32 %s
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits