[PATCH] D64033: Add column info for inline sites

[Wenlei He via Phabricator via llvm-commits]

wenlei added a comment.

In D64033#1586625 <https://reviews.llvm.org/D64033#1586625>, @dblaikie wrote:

> What's the motivation for this?


We need to differentiate inline sites on the same line for an internal (basically the tool needs to annotate each instruction with stack of inline site locations). And the missing column info for inline sites gets in the way. I thought the change will be helpful to others as well..

> Should it be conditional on "-gcolumn-info"? (or skipping it in general if the column is zero? that'd make "-gno-column-info" fall out naturally)
>  (we don't put decl_column on subprogram DIEs, for instance - admittedly, it's more likely the same function could have multiple inline call sites on the same line than you'd have multiple functions defined on the same line)

Good point. Thanks David. I will send a separate patch to skip zero column so we won't have column when "-gno-column-info" is used.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64033/new/

https://reviews.llvm.org/D64033