This doesn't show up today as we don't emit decalration only variables. This
will be tested when the followup patches implementing import of forward
declared entities lands in clang.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
This sounds like we'd want DIEs for global variable declarations or have I missed something?
It's not that we want declarations for all globals, but sometimes a declaration is all we have:
namespace ns { extern int A; } using ns::A; int foo(int Arg = A) { return Arg; }
In the above, if we want to emit the imported_declaration or the default argument, we want to have a declaration only DIE for the global.
Ah right. OK with the testing coming in short order via the rest of it as we get patches :)
This commit seems to have broken the gdb test suite:
http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/17377/steps/gdb-75-check/logs/stdio http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/17377/steps/gdb-75-check/logs/stdio
If someone with access to gdb test suite could help me figure it out, that would be great.
Fred
You can check out the testcases yourself from svn and take a look at the
differences in the 3 failing tests as far as debug info. They're short.
-eric