<div dir="ltr">ah, cool</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 27, 2016 at 7:14 PM, Dehao Chen <span dir="ltr"><<a href="mailto:danielcdh@gmail.com" target="_blank">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">Sorry that I didn't add more context about the update.<br>
<br>
Originally in this patch, I only reset discriminator when there is<br>
line change. But turns out the correct behavior is to update the<br>
discriminator whenever a special opcode has been encountered, no<br>
matter there is line change or not. This means that the implementation<br>
in the original patch is incorrect, that's why I updated the patch and<br>
added a new testcase to cover this situation.<br>
<br>
Thanks,<br>
Dehao<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Apr 27, 2016 at 6:48 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Apr 27, 2016 at 6:28 PM, Dehao Chen via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> danielcdh updated this revision to Diff 55367.<br>
>> danielcdh added a comment.<br>
>><br>
>> Fixed another bug in the implemantation to make it consistent with gnu<br>
>> assembler.<br>
><br>
><br>
> Sounds like it should be a separate patch/independent change?<br>
><br>
>><br>
>><br>
>> PTAL, Thanks<br>
>><br>
>><br>
>> <a href="http://reviews.llvm.org/D19522" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19522</a><br>
>><br>
>> Files:<br>
>>   lib/DebugInfo/DWARF/DWARFDebugLine.cpp<br>
>>   lib/MC/MCDwarf.cpp<br>
>>   test/DebugInfo/X86/discriminator2.ll<br>
>>   test/DebugInfo/X86/discriminator3.ll<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>
>><br>
><br>
</div></div></blockquote></div><br></div>