[PATCH] D40033: [NVPTX] Initial adaptation of MCAsmStreamer/MCTargetStreamer for debug info in Cuda.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: test/CodeGen/Mips/mips16ex.ll:6
 ;16-NEXT: .cfi_personality
+;16: [[TMP:.*]]:
+;16-NEXT: $func_begin0 = ([[TMP]])
----------------
echristo wrote:
> For the record this change is the one I'm concerned about. I don't think it's desired to put the cfi directives before the symbol.
> 
> That said, I'm going to have to take a closer look.
Yes, I understand this. It is required in PTX to have `.loc` before `$func_begin` symbol, otherwise it emits incorrect debug info. At least, we can make this change only for Cuda.


https://reviews.llvm.org/D40033