<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 1, 2016 at 2:39 PM, Dehao Chen <span dir="ltr"><<a href="mailto:danielcdh@gmail.com" target="_blank" class="cremed">danielcdh@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>
  call void bitcast (void (%"class.llvm::SmallVectorImpl"*)* @_ZN4llvm15SmallVectorImplIjED2Ev to void (%"class.llvm::SmallVector"*)*)(%"class.llvm::SmallVector"* %3) #7, !dbg !4175<br>
<br>
I just spent ~2 hours to generate a small C++ file to produce this "call with bitcast" pattern, but whatever I tried, front end will not give me exactly the same pattern. Shall I just keep the record in the commit log about how to reproduce with FoldingSet.bc?<br></blockquote><div><br></div><div>Not really.  Just wanted a motivation for the change.  Adding a pass as a pre-requisite for the sampler profiler means adding compile time.  Do you think this will trigger a lot in general?  I'm somewhat concerned about compile time inflation, but if it's useful in general for the sample profiling case, then I don't mind.</div><div> </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
================<br>
Comment at: test/Transforms/SampleProfile/cov-zero-samples.ll:3<br>
@@ -2,3 +2,3 @@<br>
 ;<br>
-; CHECK: remark: cov-zero-samples.cc:9:25: Applied 404065 samples from profile (offset: 2.1)<br>
+; CHECK: remark: cov-zero-samples.cc:9:29: Applied 404065 samples from profile (offset: 2.1)<br>
 ; CHECK: remark: cov-zero-samples.cc:10:9: Applied 443089 samples from profile (offset: 3)<br>
----------------<br>
</span><span class="">dnovillo wrote:<br>
> Why did the discriminators change here?  Was this because of instcombine?  What did it do?<br>
</span>This is not changing discriminator but column number.<br>
inst-combine optimized the code to combine some instructions together and used the LOC of one instruction as the new instruction's LOC.<br></blockquote><div><br></div><div>Ah, thanks.</div><div><br></div><div><br></div><div>Diego.</div></div></div></div>