[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