I'd have thought it'd be faster too.<br><br><div class="gmail_quote">On Tue Oct 21 2014 at 3:31:04 PM Diego Novillo <<a href="mailto:dnovillo@google.com">dnovillo@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 21, 2014 at 6:24 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: lib/Transforms/Scalar/SampleProfile.cpp:647<br>
@@ -645,2 +646,3 @@<br>
+                                          const LLVMContext &Ctx) {<br>
   NamedMDNode *CUNodes = F.getParent()->getNamedMetadata("<a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a>");<br>
   if (CUNodes) {<br>
----------------<br>
Why do we need this code at all? Should we just remove it in favor of the other code?<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>I was actually going to ask you.  It seemed to me that this variant might be slightly more efficient than the one that does not rely on <a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a>?</div><div>(all the insn iteration, I mean).</div><div><br></div><div>If you don't  think that's the case, then it certainly is simpler if I just remove it altogether.</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div><br></div><div>Diego.</div></div></div></div></blockquote></div>