[PATCH] [PATCH] Stop resetting NoFramePointerElim in TargetMachine::resetTargetOptions

[Eric Christopher]

We can probably make the assumption that front ends do that these days
since, of course, there aren't a lot of public front ends and clang has
been doing this for a while.

-eric

On Mon, May 18, 2015 at 3:04 PM Akira Hatanaka <ahatanak at gmail.com> wrote:

> Thanks, I'll commit this patch today unless there are further comments.
>
> We can remove NoFramePointerElimOverride if we don't need the cl::opt
> option. We can also remove NoFramePointerElim if we can assume front-ends
> or JITs today emit function attribute "no-frame-pointer-elim" to disable
> frame pointer elimination rather than setting NoFramePointerElim, but I
> don't think that is the case.
>
>
> http://reviews.llvm.org/D9830
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150518/2ef98bb9/attachment.html>