ABataev added a comment.

In https://reviews.llvm.org/D54764#1304606, @jdenny wrote:

> In https://reviews.llvm.org/D54764#1304573, @ABataev wrote:
>
> > I'm fine with the patch,
>
>
> Thanks for the quick reviews.  I'll push soon.
>
> > the original intention was to handle a possible situation where we may have 
> > several debug locations with the same line numbers.
>
> We were afraid of that.  If we find we really need this use case, we'll have 
> to find a nicer way to express it.  For reference, this use case is discussed 
> under S1 here:
>
> http://lists.llvm.org/pipermail/llvm-dev/2018-May/123258.html


Ok, got it. Thanks for the patch!


Repository:
  rC Clang

https://reviews.llvm.org/D54764



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to