<p dir="ltr"><br>
On Nov 6, 2015 4:52 PM, "Paul Robinson via llvm-commits" <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> probinson added a comment.<br>
><br>
> In <a href="http://reviews.llvm.org/D14464#284332">http://reviews.llvm.org/D14464#284332</a>, @dblaikie wrote:<br>
><br>
> > Even with column info, its insufficient if both calls came from the same<br>
> >  macro use:<br>
> ><br>
> > #define X f(); f()<br>
> >  int main() {<br>
> ><br>
> >   X;<br>
> ><br>
> > }<br>
> ><br>
> > Both calls to 'f' will have the same line and column, the location of the X<br>
> >  use.<br>
><br>
><br>
> Of course if the column numbers are the same, you need the discriminator. The provided test case did not have that situation, which is why I asked.  You don't want to be chewing up extra space in the line table if you don't need to.</p>
<p dir="ltr">Sure, agreed. Not sure if the current discriminators implementation already has the smarts to dtrt whether column info is enabled or not. (Where 'the right thing' is what you said: only adding discriminators when needed)</p>
<p dir="ltr">><br>
><br>
> <a href="http://reviews.llvm.org/D14464">http://reviews.llvm.org/D14464</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">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</p>