[PATCH] D67236: [MC] Fix undefined behavior in MCInstPrinter::formatHex

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 13:35:23 PDT 2019


dblaikie added a comment.

I'm confused - the patch description mentions passing unsigned min to a function that takes a signed value - if there's UB, it's in the caller so can't really be fixed in the callee, right? (or is the UB still present with passing int32 max too?)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67236





More information about the llvm-commits mailing list