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

[Hans Wennborg via cfe-dev]

Merged it in r292339.

Cheers,
Hans

On Tue, Jan 17, 2017 at 12:19 PM, Jonathan Coe <jbcoe at me.com> wrote:
> 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
>>
>>
>