[PATCH] specify dwarf version for Solaris

David Blaikie dblaikie at gmail.com
Wed Oct 15 10:22:31 PDT 2014


LGTM, please commit.

(I wouldn't mind it if there were some sense that this was a decision made
by the Solaris platform - perhaps you are such a spokesperson for Solaris,
but I don't know)

On Wed, Oct 15, 2014 at 10:10 AM, Alexander Eremin <
alexander.r.eremin at gmail.com> wrote:

> Thanks for pointing, this is updated patch.
>
> Alex
>
> 2014-10-15 19:24 GMT+04:00 David Blaikie <dblaikie at gmail.com>:
>
>> Sorry, I'm not sure I follow - what I meant is the patch needs (or should
>> have, unless there's a reason this isn't testable, etc) a test case in
>> Clang's regression suite. Probably in clang/test/CodeGen/debug-info-*
>>
>> (you could try removing other targets from that conditional and see which
>> tests fail - that should piont you to where to add tests for this platform)
>>
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141015/ff223913/attachment.html>


More information about the cfe-commits mailing list