[PATCH] D90539: Make CallInst::updateProfWeight emit i32 weights instead of i64
Mircea Trofin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 3 19:29:21 PST 2020
mtrofin added a comment.
In D90539#2372744 <https://reviews.llvm.org/D90539#2372744>, @aeubanks wrote:
> Ideally they'd be i64, see https://reviews.llvm.org/D88609 where I tried doing that, but it was reverted multiple times due to uint64_t overflows and I gave up trying to fix the various issues. There are multiple instances of using uint64_t to do uint32_t arithmetic and checking for overflows, and I eventually got annoyed with all those and gave up.
Ah... iiuc, with 32 bit, overflow can be detected more easily?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90539/new/
https://reviews.llvm.org/D90539
More information about the llvm-commits
mailing list