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