[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