[PATCH] Do not split variables definitions into 2 DIEs.

[Eric Christopher]

On Thu, Oct 9, 2014 at 4:19 PM, Frederic Riss <friss at apple.com> wrote:
> I finally managed to get some test results for this patch. Indeed putting the static member variable definition broke some lldb tests, thus I left the behavior unchanged for static member variables. This is still a nice cleanup of the global variable DIE creation logic.

Curious what broke?

-eric

>
> David, if you still like the direction I'd appreciate if you could give it a test run on the GDB testsuite to see if it doesn't break anything there. Note that I haven't included an additional test for type units as I removed the behavior change that might have caused issues with that usecase.
>
> http://reviews.llvm.org/D5457
>
> Files:
>   lib/CodeGen/AsmPrinter/DwarfUnit.cpp
>   test/DebugInfo/X86/gnu-public-names.ll