[PATCH] D56587: Introduce DW_OP_LLVM_convert

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 13 06:19:07 PST 2019


probinson added a subscriber: ABataev.
probinson added a comment.

+ @ABataev re the question whether NVPTX runs into the situation described in this review.

The Sony debugger guys are okay with using the GCC operator in a pre-v5 expression. So, tentatively, for all debugger tunings, we can emit that instead of the more complicated expression.  That way we are emitting compliant expressions, and the info doesn't just disappear sometimes (a much worse outcome IMO).  The only remaining question is my hypothetical about NVPTX.

Re branch operators, I thought Adrian was throwing that out there as a general concern; yes branch operators exist, and yes we don't use them currently. As David says, the assembler knows how to convert a label difference into a ULEB and it will all Just Work.  If/when we ever need it to.


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

https://reviews.llvm.org/D56587





More information about the llvm-commits mailing list