[cfe-dev] Issues with https://reviews.llvm.org/D26082

[Jonathan Coe via cfe-dev]

I've raised https://llvm.org/bugs/show_bug.cgi?id=31673 to track the
version number check fix.

I think this should be merged to the 4.0 branch.

regards,

Jon

On 13 January 2017 at 16:38, Jonathan Coe <jbcoe at me.com> wrote:

>
>
> On 13 January 2017 at 15:37, Renato Golin <renato.golin at linaro.org> wrote:
>
>> On 13 January 2017 at 15:31, Jonathan Coe <jbcoe at me.com> wrote:
>> > Added https://llvm.org/bugs/show_bug.cgi?id=31628.
>> >
>> > Would be cool if you could sanity check it. I'm still new to this.
>>
>> Sure! Please add the reviews to Phabricator
>> (https://reviews.llvm.org/differential/) and I'll review.
>>
>>
> I've reverted the problematic patch and added https://reviews.llvm.
> org/D28682 to fix trivial language incompatibilities and fail if Python
> version !=2.
>
> regards,
>
> Jon
>
>
>> cheers,
>> --renato
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20170117/74ae628d/attachment.html>