testcase (and for r227215)? :-) On 27 January 2015 at 18:08, Filipe Cabecinhas <[email protected]> wrote:
> Author: filcab > Date: Tue Jan 27 12:08:32 2015 > New Revision: 227219 > > URL: http://llvm.org/viewvc/llvm-project?rev=227219&view=rev > Log: > Fix part of r227215. PS4 code just omits leaf frame pointers. > > Modified: > cfe/trunk/lib/Driver/Tools.cpp > > Modified: cfe/trunk/lib/Driver/Tools.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=227219&r1=227218&r2=227219&view=diff > > ============================================================================== > --- cfe/trunk/lib/Driver/Tools.cpp (original) > +++ cfe/trunk/lib/Driver/Tools.cpp Tue Jan 27 12:08:32 2015 > @@ -2306,12 +2306,10 @@ static bool shouldUseFramePointerForTarg > case llvm::Triple::systemz: > case llvm::Triple::x86: > case llvm::Triple::x86_64: > - if (Triple.isOSLinux()) { > + if (Triple.isOSLinux()) > if (Arg *A = Args.getLastArg(options::OPT_O_Group)) > if (!A->getOption().matches(options::OPT_O0)) > return false; > - } else if (Triple.isPS4CPU()) > - return false; > return true; > case llvm::Triple::xcore: > return false; > @@ -2335,6 +2333,9 @@ static bool shouldUseLeafFramePointer(co > options::OPT_momit_leaf_frame_pointer)) > return > A->getOption().matches(options::OPT_mno_omit_leaf_frame_pointer); > > + if (Triple.isPS4CPU()) > + return false; > + > return shouldUseFramePointerForTarget(Args, Triple); > } > > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
