[all-commits] [llvm/llvm-project] fffd70: [LLDB] Replacing use of ul suffix in GetMaxU64Bitf...

Shafik Yaghmour via All-commits all-commits at lists.llvm.org
Thu Dec 5 10:04:03 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: fffd70291e124efc4a5bd475dccfc41cbc4bca8a
      https://github.com/llvm/llvm-project/commit/fffd70291e124efc4a5bd475dccfc41cbc4bca8a
  Author: shafik <syaghmour at apple.com>
  Date:   2019-12-05 (Thu, 05 Dec 2019)

  Changed paths:
    M lldb/source/Utility/DataExtractor.cpp
    M lldb/unittests/Utility/DataExtractorTest.cpp

  Log Message:
  -----------
  [LLDB] Replacing use of ul suffix in GetMaxU64Bitfield since it not guarenteed to be 64 bit

GetMaxU64Bitfield(...) uses the ul suffix but we require a 64 bit unsigned integer and ul could be 32 bit. So this replacing it with a explicit cast and refactors the code around it to use an early exit.

Differential Revision: https://reviews.llvm.org/D70992




More information about the All-commits mailing list