r227219 - Fix part of r227215. PS4 code just omits leaf frame pointers.

[Rafael Espíndola]

testcase (and for r227215)? :-)

On 27 January 2015 at 18:08, Filipe Cabecinhas <me at filcab.net> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150128/0e649d44/attachment.html>