r227343 On Wed, Jan 28, 2015 at 6:00 PM, Rafael Espíndola < [email protected]> wrote:
> 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 > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
