<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 26, 2016 at 1:58 PM, Dehao Chen via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Unfortunately, we can't because I need to have two consecutive<br>
instructions that are both from discriminator 1, but different lines<br>
to trigger this bug. With your proposed change, the code sequence will<br>
be:<br>
<br>
line 4 discriminator 0<br>
line 4 discriminator 1<br>
line 5 discriminator 0<br>
line 5 discriminator 1<br>
<br>
But with my test case, the code sequence will be:<br>
<br>
line 4 discriminator 0<br>
line 5 discriminator 0<br>
line 4 discriminator 1<br>
line 5 discriminator 1<br></blockquote><div><br></div><div>Ah, OK - thanks for the description (might be worth including that in the test case, but I'm not sure - up to you)<br><br>Side note: Are we not including the column info when determining when to add discriminators? Is there a reason for that (I vaguely recall asking this before, so I assume we are intentionally disregarding column info for some good reason I've forgotten). Because these locations are all unique once you include column info (which we are including by default now).<br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On Tue, Apr 26, 2016 at 12:46 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> dblaikie added inline comments.<br>
><br>
> ================<br>
> Comment at: test/DebugInfo/X86/discriminator2.ll:9<br>
> @@ +8,3 @@<br>
> +; #3 void baz() {<br>
> +; #4   foo/*discriminator 1*/(bar(),<br>
> +; #5       bar());bar()/*discriminator 1*/;<br>
> ----------------<br>
> Would it be simpler/sufficient to do this:<br>
><br>
>   void f1();<br>
>   void f2() {<br>
>     f1(); f1();<br>
>     f1(); f1();<br>
>   }<br>
><br>
> (with column info off? Or, to keep column info on, #define CALLS f1(); f1() and have the body of the function be:<br>
>   CALLS;<br>
>   CALLS;<br>
> )<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D19522" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19522</a><br>
><br>
><br>
><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>