[llvm-commits] [llvm-gcc-4.2] r62109 - in /llvm-gcc-4.2/trunk/gcc: llvm-debug.cpp llvm-debug.h

[Chris Lattner]

On Jan 12, 2009, at 2:27 PM, Devang Patel wrote:

> Author: dpatel
> Date: Mon Jan 12 16:27:00 2009
> New Revision: 62109
>
> URL: http://llvm.org/viewvc/llvm-project?rev=62109&view=rev
> Log:
> Do not generate multiple compile_unit, it has many issues in current  
> implementation. The code generator is not set up to handle multiple  
> complete or partial compile_units as per dwarf specification. On  
> darwin, the tool chain expects one compile_unit per .o file.

FWIW, I completely agree with this change.  I think it makes a lot  
more sense to focus on being as good as GCC's debug info (even if  
buggy and not optimal) and then try to improve on it as a separate  
step.  Trying to represent headers as their own compile units sounds  
great, but causes lots of problems in practice.

-Chris