<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 4, 2016 at 6:51 PM Dehao Chen <<a href="mailto:danielcdh@gmail.com">danielcdh@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">danielcdh added inline comments.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
> probinson wrote in AddDiscriminators.cpp:165<br class="gmail_msg">
> Why analyze the CFG to create discriminators that you are not going to emit?  What value does that have?  Seems like a waste of time.<br class="gmail_msg">
<br class="gmail_msg">
For SamplePGO, it uses debug info including discriminator to annotate CFG. So even when the discriminator is not emitted, it needs to be used by the profile annotator during compilation.<br class="gmail_msg"></blockquote><div><br></div><div>I don't fully understand that - but I'll leave that to Diego to crosscheck.<br><br>If that's the case, then, yes - we probably want a way to support adding discriminators for SamplePGO independently of adding (or at least using) discriminators for DWARF.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25132" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25132</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>