[PATCH] D46189: [DEBUGINFO, NVPTX] Enable support for the debug info on NVPTX target.

[Haojian Wu via Phabricator via llvm-commits]

hokein added a comment.

This patch may generate illegal assembly code, see the reproduced commands below.

  $ ./bin/clang -cc1 -triple nvptx64-nvidia-cuda -aux-triple x86_64-grtev4-linux-gnu -S -disable-free -disable-llvm-verifier -discard-value-names -main-file-name new.cc -mrelocation-model pic -pic-level 2 -mthread-model posix -fmerge-all-constants -mdisable-fp-elim -relaxed-aliasing -no-integrated-as -mpie-copy-relocations -munwind-tables -fcuda-is-device -target-feature +ptx60 -target-cpu sm_35 -dwarf-column-info -debug-info-kind=line-directives-only -dwarf-version=2 -debugger-tuning=gdb -o empty.s -x cuda empty.cc
  $  cat empty.s
  //
  // Generated by LLVM NVPTX Back-End
  //
  
  .version 6.0
  .target sm_35
  .address_size 64
  
  
  
  	}

I think the bug existed even before this patch, we have an extra `// }`, but the code is legal to compile.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D46189