[PATCH] D54200: [PPC64] Use -0x80000000LL instead of std::numeric_limits<int32_t>::min()

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 7 03:51:50 PST 2018


arichardson added a comment.

I agree with James that numeric_limits is clearer. I've run into bugs before because there was a 0/f missing when trying to use max/min value integers.

Is the problem here that we are missing a cast to int64_t?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D54200





More information about the llvm-commits mailing list