<div class="gmail_quote">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>Looks good. I left a minor comment to spruce up a test.</div><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><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><span class="HOEnZb"><font color="#888888"><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>

<br>
</font></span></div>
</blockquote></div><br>