[PATCH] D148150: [InstrProf][Temporal] Add weight field to traces

Ellis Hoag via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 11:33:36 PDT 2023


ellis added inline comments.


================
Comment at: llvm/unittests/ProfileData/InstrProfTest.cpp:252
 
-  Writer.addTemporalProfileTraces({LargeTrace, SmallTrace}, 2);
+  SmallVector Traces = {LargeTrace, SmallTrace};
+  Writer.addTemporalProfileTraces(Traces, 2);
----------------
This broke some builds because of compiler warnings, but I believe it should be fixed by https://reviews.llvm.org/D148259.

https://lab.llvm.org/buildbot/#/builders/57/builds/26047

```
/home/buildbots/docker-RHEL84-buildbot/SetupBot/worker_env/ppc64le-clang-rhel-test/clang-ppc64le-rhel/llvm-project/llvm/unittests/ProfileData/InstrProfTest.cpp:252:3: error: 'SmallVector' may not intend to support class template argument deduction [-Werror,-Wctad-maybe-unsupported]
  SmallVector Traces = {LargeTrace, SmallTrace};
  ^
/home/buildbots/docker-RHEL84-buildbot/SetupBot/worker_env/ppc64le-clang-rhel-test/clang-ppc64le-rhel/llvm-project/llvm/include/llvm/ADT/SmallVector.h:1199:22: note: add a deduction guide to suppress this warning
class LLVM_GSL_OWNER SmallVector : public SmallVectorImpl<T>,
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148150



More information about the llvm-commits mailing list