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

Eric Christopher echristo at gmail.com
Thu Oct 9 17:06:25 PDT 2014

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?


> 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

More information about the llvm-commits mailing list