<br><br><div class="gmail_quote">On Wed, Dec 28, 2011 at 12:35 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">On Wed, Dec 28, 2011 at 2:06 AM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>> I believe the best option for us is to add the<br>
> --fno-omit-frame-pointer depending on the -faddress-sanitizer flag.<br>
> But in order to do this reliably, we need to remove all the instances<br>
> of -fomit-frame-pointer from the command line.<br>
</div>Why? Just add -fno-omit-frame-pointer last.<br></blockquote><div><br></div></div><div>Yea, this should be easy to do inside the Clang driver. Feel free to CC me on any patch, and send i to cfe-commits@</div></div></blockquote>
<div><br></div><div>One thing in favor of documenting the need for -fno-omit-frame-pointer instead of enforcing it in the driver: </div><div>suppose a program, part of which is compiled with -faddress-sanitizer, but some other part is compiled w/o it. </div>
<div>If -fno-omit-frame-pointer is implied by -faddress-sanitizer and not explicitly given by users, some part of the program will not have frame pointers and stacks will be short.</div><div>I am inclined to document the need to -fno-omit-frame-pointer. </div>
<div>Other opinions? </div><div><br></div><div>--kcc </div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br>