[PATCH] D41827: [DEBUG] Initial adaptation of NVPTX target for debug info emission.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 11:13:47 PST 2018


ABataev added a comment.

In https://reviews.llvm.org/D41827#1007783, @echristo wrote:

> In https://reviews.llvm.org/D41827#1007515, @ABataev wrote:
>
> > Ping
>
>
> At this point I'm looking at how to break this up for you. It's going to take more time because I have other things, but I don't want to put in all of this in a single patch because there are things I want to reason about separately.
>
> -eric


Hi Eric, thanks for your help. My ping is not a ping for the review, it is a ping for my question.
It is very hard to split this patch. It will break the generation of the PTX files so that either the compiler will crash or ptxas won't be able to compile the generated file. I asked you, is this ok if as the first part of the patch I remove generation of the debug lines info? It will reduce the size of the patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D41827





More information about the llvm-commits mailing list