[PATCH] D56587: Introduce DW_OP_LLVM_convert

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 14:22:41 PST 2019


dblaikie added a comment.

In D56587#1393746 <https://reviews.llvm.org/D56587#1393746>, @probinson wrote:

> In D56587#1393556 <https://reviews.llvm.org/D56587#1393556>, @dblaikie wrote:
>
> > I don't think anyone mentioned it, that's why I was bringing it up - there's always an option to not render any location if it's not possible/worth the work. That's all I was asking - is it worth the complexity? (I wasn't sure anyone needed it - but sounds like Sony does, reckon it's worth the tradeoff in complexity in LLVM compared to the work required to support this in the Sony debugger?)
>
>
> NVPTX also would need it, because they are stuck on DWARF v2.


Any ideas if NVPTX hit this case? my understanding was that NVPTX has a fairly restrictive set of code or actions that can be used.


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

https://reviews.llvm.org/D56587





More information about the llvm-commits mailing list