[PATCH] D53821: [ELF][PPC64]Workaround bogus Visual Studio build warning

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 29 09:56:15 PDT 2018


sfertile accepted this revision.
sfertile added a comment.
This revision is now accepted and ready to land.

Thanks for cleaning this up. LGTM.

> We could also fix this by using INT32_MIN or std::numeric_limts<int32_t>::min(), if preferred

Personally I prefer either of those over the raw numeric literal.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D53821





More information about the llvm-commits mailing list