[PATCH] D131869: [llvm] Remove std::clamp equivalent in `Transforms/Utils/MisExpect.cpp`

Joe Loser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 17 08:40:44 PDT 2022


jloser marked 2 inline comments as done.
jloser added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/MisExpect.cpp:75
 
 uint64_t getMisExpectTolerance(LLVMContext &Ctx) {
   return std::max(static_cast<uint64_t>(MisExpectTolerance),
----------------
paulkirth wrote:
> Ugh. in D131935 I think I forgot to update the return type here, despite changing everything else to `uint32_t`. 
> 
> When you rebase, could you update this?
Yeah, I noticed this. Not a biggie - just fixed and rebased.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131869/new/

https://reviews.llvm.org/D131869



More information about the llvm-commits mailing list