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@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@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
