[PATCH] D27712: [ELF] - Do not crash when move location counter backward.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 23:41:30 PST 2016


grimar added a comment.

In https://reviews.llvm.org/D27712#623084, @ruiu wrote:

> LGTM. I'm not convinced that https://reviews.llvm.org/D25279 is worth its complexity


May be. Though I think we should be able to replace uint64_t Dot to "some checked type" Dot,
and that can solve all possible overflow issues once with just a this local change.
It looks convinent to have such overflow detector class, may be we can have one in llvm. Not sure here.


Repository:
  rL LLVM

https://reviews.llvm.org/D27712





More information about the llvm-commits mailing list