[llvm] r244319 - [dsymutil] Use the new MCDwarfLineTableParams customization to emit linetables

Frédéric Riss via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 10:33:39 PDT 2015


I tried a fix in r244327 (I’m supposing these platforms use an unsigned char by default).

If this doesn’t work, feel free to revert (you’ll need to revert 244318 and which depends on it 244319). I’m watching the bots, but I need to go to a meeting right now.

Fred


> On Aug 7, 2015, at 10:28 AM, Renato Golin <renato.golin at linaro.org> wrote:
> 
> 
> Looks like they're all off-by-one line numbers. This also caused the
> Sanitizer errors that we're seeing in other buildbots.
> 
> 
> 
> On 7 August 2015 at 18:23, Renato Golin <renato.golin at linaro.org> wrote:
>> On 7 August 2015 at 16:14, Frederic Riss via llvm-commits
>> <llvm-commits at lists.llvm.org> wrote:
>>> Author: friss
>>> Date: Fri Aug  7 10:14:13 2015
>>> New Revision: 244319
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=244319&view=rev
>>> Log:
>>> [dsymutil] Use the new MCDwarfLineTableParams customization to emit linetables
>>> 
>>> llvm-dsymutil has to be able to process debug info produced by other compilers
>>> which use different line table settings. The testcase wasn't generated by
>>> another compiler, but by a modified clang.
>> 
>> Hi Frederic,
>> 
>> One of your two patches may have broken some tests on our bot:
>> 
>> http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/6952
>> 
>> cheers,
>> --renato



More information about the llvm-commits mailing list