[PATCH] D88609: Use uint64_t for branch weights instead of uint32_t
Arthur Eubanks via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 28 13:17:47 PDT 2020
aeubanks added a comment.
This was reverted due to crashes caused by adding branch weights overflowing uint64_t. Now in BranchProbabilityInfo.cpp, if the sum of weights overflows, scale down all weights by UINT32_MAX (See `ScaleWeights()`). Is that ok?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88609/new/
https://reviews.llvm.org/D88609
More information about the cfe-commits
mailing list