[PATCH] D15977: [Clang] Supporting all entities declared in lexical scope in LLVM debug info

[David Blaikie via cfe-commits]

dblaikie added a comment.

Looks good - though you could do that one further simplification to the test case, I think. (removing x/if x)


================
Comment at: test/CodeGenCXX/debug-info-lb.cpp:5
@@ +4,3 @@
+  static int bar = 1;
+  if (x)
+  {
----------------
Looks like you could remove this "if (x)" and the "int x" parameter & still keep the test otherwise in tact.


http://reviews.llvm.org/D15977