[PATCH] D63325: [Support][Time profiler] Make FE codegen blocks to be inside frontend blocks

Fri Aug 23 11:37:08 PDT 2019

anton-afanasyev added a comment.

In D63325#1643023 <https://reviews.llvm.org/D63325#1643023>, @nathanchance wrote:

> Done, thanks for looking into this!
>
> F9847579: check-time-trace-sections.json <https://reviews.llvm.org/F9847579>

Hmm, I still didn't manage to reproduce the issue using your json-file:

  [~/llvm] > cat check-time-trace-sections.json | python llvm-project/clang/test/Driver/check-time-trace-sections.py
  [~/llvm] > echo $status
  0

The file itself looks good for this test: all Codegen sections are inside any Frontend section and all Frontend sections are before all Backend sections.
Also your log shows no error messages, so test should pass ok. Could it be an issue with python? What is the version you are using?

Repository:
  rL LLVM

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

https://reviews.llvm.org/D63325