aprantl added inline comments.

================
Comment at: clang/lib/CodeGen/CGDebugInfo.h:536
 
+  /// Create and attach debuginfo to a the provided string literal GV.
+  void AddStringLiteralDebugInfo(llvm::GlobalVariable *GV, StringRef Name,
----------------
I would appreciate a comment explaining the motivation behind doing this here.


================
Comment at: clang/test/CodeGen/debug-info-variables.c:6
+
+// CHECK: DIGlobalVariable({{.*}}line: [[@LINE+2]]
+const char* s() {
----------------
would be nice to also check the the type is char*, etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123534

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to