<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 24, 2014 at 4:06 PM, Xinliang David Li <span dir="ltr"><<a href="mailto:xinliangli@gmail.com" target="_blank">xinliangli@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Diego,<br><br>I think sampleFDO needs to be designed in a way which can protect itself from future breakage like this. The roots in the unnecessary dependency of sample FDO on gmlt setting. It is totally reasonable to tune debug binary size by changes like this.</blockquote><div><br></div><div>FWIW, it's not 100% clear to me that this is the right change. Might be worth chatting about on Monday.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>The right way is to fix this is to introduce an internal -g<...> flag for use by sampleFDO -- it will have a fixed definition of what needs to be emitted.</div></blockquote><div><br></div><div>This doesn't seem worth having an internal flag for. Google is probably the primary consumer of Clang's -gline-tables-only data. If it doesn't meet our needs, we can lobby to make it behave the way we need it to.</div></div></div></div>