<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 22, 2013 at 10:08 AM, Diego Novillo <span dir="ltr"><<a href="mailto:dnovillo@google.com" target="_blank" class="cremed">dnovillo@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="im">On Tue, Oct 22, 2013 at 1:06 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>

> My only objection to this patch is the use of the word "auto". ;] While I<br>
> know there is some historical precedent, I think that if this is going into<br>
> mainline Clang, it would be worthwhile to strive for a more descriptive<br>
> name. '-fsample-profile'? Other ideas?<br>
<br>
</div>It cannot be 'sample'.  This option will take all kinds of profiles<br>
that will be treated separately in the backend according to their<br>
nature.<br></blockquote><div><br></div><div>I'm surprised -- I expected this to essentially only be used with sample-based profiles? Bob seemed to have a different architecture in mind for other profile sources?</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
We need something generic here.</blockquote><div><br></div><div>If this is going to be something generic, we should have some input from Bob as well at least. And then the flag name it seems should be "-fprofile" or "-fprofile-file" or "-fprofile-input"....</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> What's the problem with "auto". Does<br>
it have a different connotation?</blockquote></div><br>There isn't actually anything automatic about it? The compiler doesn't run the program while compiling it to produce a profile and then use that to re-compile.... ;]</div>
</div>