[PATCH] D128070: [Stats] Change Value type from unsigned to uint64_t.

Mingming Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 10:02:10 PDT 2022


mingmingl created this revision.
Herald added a subscriber: hiraditya.
Herald added a project: All.
mingmingl requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Passes could store large integers into stats, as a result of using branch frequency counters.

Using uint64_t gives a much smaller chance of overflow and reduces confusion. One example pass is `block-placement-stats`.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D128070

Files:
  llvm/include/llvm/ADT/Statistic.h
  llvm/lib/Support/Statistic.cpp
  llvm/lib/Transforms/Scalar/NewGVN.cpp
  llvm/unittests/ADT/StatisticTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128070.437927.patch
Type: text/x-patch
Size: 5427 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220617/5f4eb598/attachment.bin>


More information about the llvm-commits mailing list