<br><br><div class="gmail_quote">On Tue, May 29, 2012 at 12:00 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.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><div class="h5">On Tue, May 29, 2012 at 12:46 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi!<div><br></div><div>This change makes Clang driver pass only the last option from "-g" group to the compiler:</div><div>* if it is -g0, no option is passed;</div><div>* if it is -gline-tables-only, this option is preserved;</div>


<div>* all other -g* options are substituted with -g.</div><div><br></div><div><a href="http://codereview.appspot.com/6243065/" target="_blank">http://codereview.appspot.com/6243065/</a></div></blockquote><div><br></div>
</div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div>Looks good. I left a minor comment to spruce up a test.</div>
</div></blockquote><div><br></div><div>Submitted as r157602, thanks!</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><br></div><div>
The "This is wrong." comment makes me sad. We should eventually do something along the lines of the following:</div><div><br>
</div><div>1) for all the gdb & dwarf variations collapse them to just '-g'.</div><div>2) support things like -gtoggle</div><div>3) ignore a bunch of the flag options (-gstrict-dwarf, -grecord-gcc-switches, etc)</div>

<div>4) produce a driver error on -gstabs, -gcoff, -gvms, etc. All the formats that Clang/LLVM doesn't support</div><div><br></div><div>We could leave a better fixme, or if you're feeling generous, you could work on this? It shouldn't be a lot of work...</div>
</div></blockquote><div><br></div><div>I see. For this patch, I've just added these items to fixme, and will try to fix them later this week.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><span><font color="#888888"><br clear="all"><span class="HOEnZb"><font color="#888888"><div>
<br></div>-- <br><div>Alexey Samsonov, MSK</div>

<br>
</font></span></font></span></div>
</blockquote></div><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>