[patch] Enable column information by default

Eric Christopher echristo at gmail.com
Wed Jul 9 18:37:01 PDT 2014


He did, though he didn't mention it. Approximately 0.

-eric

On Wed, Jul 9, 2014 at 6:35 PM, Nico Weber <thakis at chromium.org> wrote:
> Did you measure the compile time impact of this?
>
> On Jul 9, 2014 6:29 PM, "Diego Novillo" <dnovillo at google.com> wrote:
>>
>> This patch implements my proposal to turn column information by
>> default. Some background and discussion at:
>>
>> http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-June/074290.html
>>
>> David, as we discussed offline, I did not see any regressions in the
>> gdb testsuite on gdb's trunk. I will try with the instructions you
>> gave me for gdb 7.5 tomorrow.
>>
>> This also updates the documentation and some tests that relied on the
>> lack of column information. Some tests had column information in the
>> expected output, but it was wrong (the tsan tests). Others were using
>> the driver to execute.
>>
>> Tested on x86_64. Anything else I should test?
>>
>>
>> Thanks.  Diego.
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>



More information about the cfe-commits mailing list