hctim marked an inline comment as done.
hctim added inline comments.

================
Comment at: clang/test/CodeGen/debug-info-variables.c:1
+// RUN: %clang_cc1 %s -debug-info-kind=standalone -S -emit-llvm -o - | 
FileCheck %s
+
----------------
dblaikie wrote:
> Presumably named variables are already tested elsewhere - so I probably 
> wouldn't test them here, and would give this file a more specific name 
> related to string literals. Alternatively/perhaps the other thing to do would 
> be to find some test that already covers global variable debug info, and test 
> string literals there?
I found `debug-info-global-constant.c` and `debug-info-static.c`, but neither 
of them test DILocalVariables or strings.

This test was cargo-culted from `debug-info-gline-tables-only.c` which does the 
inverse :).

I was hoping that this test would become the main "simple global variables" 
test :)


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