[PATCH] D69056: [LNT] Remove Python2 support

Fri Sep 3 03:30:50 PDT 2021

tnfchris added a comment.

In D69056#2981892 <https://reviews.llvm.org/D69056#2981892>, @thopre wrote:

> In D69056#2981873 <https://reviews.llvm.org/D69056#2981873>, @tnfchris wrote:
>
>> In D69056#2981864 <https://reviews.llvm.org/D69056#2981864>, @thopre wrote:
>>
>>> In D69056#2981853 <https://reviews.llvm.org/D69056#2981853>, @tnfchris wrote:
>>>
>>>> Ohh it's happening!
>>>>
>>>> This looks good but you've missed the cPerf stuff, https://github.com/llvm/llvm-lnt/blob/f48d431f44610e339d00a33d57564c6029c4ff43/lnt/testing/profile/cPerf.cpp#L827 and lnt/testing/profile/__init__.py
>>>
>>> What's Python2 specific in lnt/testing/profile/__init__.py?
>>
>> The futures import at the top https://github.com/llvm/llvm-lnt/blob/f48d431f44610e339d00a33d57564c6029c4ff43/lnt/testing/profile/__init__.py#L4
>
> It's already there: https://reviews.llvm.org/D69056#change-H7ZXp1LFQQlR

Oh I had missed it :D

Repository:
  rLNT LNT

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69056/new/

https://reviews.llvm.org/D69056