[llvm-commits] [llvm] r103135 - /llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp

[Devang Patel]

On Thu, May 6, 2010 at 12:14 PM, Devang Patel <devang.patel at gmail.com> wrote:
> Dan,
>
> On Thu, May 6, 2010 at 11:13 AM, Devang Patel <devang.patel at gmail.com> wrote:
>> If the tools (compiler, debugger, performance analyzer, ...) can
>> provide better end user experience by saying "this instruction has
>> unknown location" instead of "I do not have location info for this
>> instruction" then I'm all for it. If not then we're doing extra work,
>> expecting extra work from other tools, bloating line tables.
>
> At -O3, line table for SelectionDagBuilder.cpp grow by more then 100%
> after this change.
>
> BTW, DWARF standard does not mandate that each instruction must have
> associated location information.
>

And this patch causes regression gdb test suite (pc-verify.exp).
Please revert this and subsequent patches.
-
Devang