[PATCH] D90539: Make CallInst::updateProfWeight emit i32 weights instead of i64
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 3 19:25:00 PST 2020
aeubanks added a comment.
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.
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