This revision was automatically updated to reflect the committed changes.
Closed by commit rL370300: [Test][Time profiler] Fix test for python3 (authored
by anton-afanasyev, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://r
nathanchance accepted this revision.
nathanchance added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66873/new/
https://reviews.llvm.org/D66873
__
anton-afanasyev added a comment.
Could you please give LGTM?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66873/new/
https://reviews.llvm.org/D66873
___
cfe-commits mailing list
cfe-commits@lists.llvm
nathanchance added a comment.
The test case passes for me after this, thanks for the fix!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66873/new/
https://reviews.llvm.org/D66873
___
cfe-commits mailin
anton-afanasyev created this revision.
anton-afanasyev added reviewers: mgehre, nathanchance.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Fix test checking time profiler generates correct tracing json-file.
`filter` works differently for python2 and python3, so unifying
