Yup. Patch is fine then. Assuming you have commit access?<br><br>-eric<br><br><div>On Fri Jan 10 2014 at 12:50:16 PM, Gaël Thomas <<a href="mailto:gael.thomas00@gmail.com">gael.thomas00@gmail.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>Exactly :) clang assumes -fomit-frame-pointer with -O3 on linux/x86_64, </p>
<p>Gaël </p>
<div>Le 10 janv. 2014 21:36, "Eric Christopher" <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> a écrit :</div><div><br type="attribution"><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

It's definitely what's expected, basically working around compilers that assume -fomit-frame-pointer by default?<br><div><br></div><div>-eric</div><br><div>On Fri Jan 10 2014 at 7:12:02 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:</div>


<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Seems reasonable. CCing Eric who is the code owner of configure/make<br>
build system.<br>
<br>
On 23 December 2013 14:56, Gaël Thomas <<a href="mailto:gael.thomas00@gmail.com" target="_blank">gael.thomas00@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> I made a small patch to modify the file Makefile.rules when the flag<br>
> REQUIRES_FRAME_POINTER is used. In this case, I have added the<br>
> -fno-omit-frame-pointer flag to force the use of the frame pointer.<br>
> It's mandatory for vmkit, which supposes that all the frames are<br>
> "well" formed in the sense that they use frame pointers (we rely on<br>
> the frame pointers to inspect the stack during a collection). As the<br>
> flag already exists, I think that forcing frame pointers is exactly<br>
> what the user could expect, even when the code is optimized?<br>
><br>
> See you,<br>
> Gaël<br>
><br>
><br>
><br>
> --<br>
> ------------------------------<u></u>------------------------------<u></u>-------<br>
> Gaël Thomas, Associate Professor, UPMC<br>
> <a href="http://pagesperso-systeme.lip6.fr/Gael.Thomas/" target="_blank">http://pagesperso-systeme.<u></u>lip6.fr/Gael.Thomas/</a><br>
> ------------------------------<u></u>------------------------------<u></u>-------<br>
><br>
> ______________________________<u></u>_________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
><br>
</blockquote>
</blockquote></div></blockquote>