[PATCH] D78153: [TimeProfiler] Fix some style issues. NFC

Russell Gallop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 03:14:44 PDT 2020


russell.gallop accepted this revision.
russell.gallop added a comment.
This revision is now accepted and ready to land.

LGTM, with assertion messages adjusted.



================
Comment at: llvm/lib/Support/TimeProfiler.cpp:124-129
     assert(Stack.empty() &&
            "All profiler sections should be ended when calling Write");
     assert(std::all_of(ThreadTimeTraceProfilerInstances.begin(),
                        ThreadTimeTraceProfilerInstances.end(),
                        [](const auto &TTP) { return TTP->Stack.empty(); }) &&
            "All profiler sections should be ended when calling Write");
----------------
Assertion messages also need adjusting. Write -> write.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78153





More information about the llvm-commits mailing list