[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:31:12 PST 2020


aeubanks added a comment.

Yeah, just do 32-bit arithmetic with uint64_t and you can see if the final result overflows. One example is here: https://github.com/llvm/llvm-project/blob/7ba3293691beb9a2c6ea4a81064c24580afe5816/llvm/lib/Analysis/BranchProbabilityInfo.cpp#L486


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