[Openmp-commits] [PATCH] D157725: [OpenMP] Fix Slice Duplicate in Profiler

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

Could you share how the json looks now?

> Having nested time regions is a normal thing.

Yes, but our nesting did never provide real benefit. We did not have good reason for the placement. Let's go back to minimal placement and then see where it makes sense to add nested timers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157725