Skip to content

Need to cleanup paddle/fluid/platform/profiler.* #9752

@wangkuiyi

Description

@wangkuiyi
  1. Functions ParseEvents and PrintProfiler are only internally used and should be removed from profiler.h:

    // Parse the event list and output the profiling report
    void ParseEvents(std::vector<std::vector<Event>>&,
    EventSortingKey sorted_by = EventSortingKey::kDefault);
    // Print results
    void PrintProfiler(std::vector<std::vector<EventItem>>& events_table,
    std::string& sorted_domain, const size_t name_width,
    const size_t data_width);

  2. EventKind is only internally used and should be renamed Event::Type.

    enum EventKind { kMark, kPushRange, kPopRange };

  3. EventList is only internally used and should be moved into profiler.cc.

    struct EventList {

  4. EventItem is only internally used and should be moved into profiler.cc

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions