[PATCH] D66411: Fix -ftime-trace breaking flame-graph assumptions

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 10:06:05 PDT 2019


mgrang added inline comments.


================
Comment at: llvm/lib/Support/TimeProfiler.cpp:103
+    auto SortedEntries = SmallVector<Entry, 128>(Entries);
+    llvm::sort(SortedEntries.begin(), SortedEntries.end(),
+               [](const Entry &A, const Entry &B) {
----------------
You can use the range-based version of llvm:sort here.

```
llvm::sort(SortedEntries);
```


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

https://reviews.llvm.org/D66411





More information about the llvm-commits mailing list