[all-commits] [llvm/llvm-project] 0e1fb4: [lld-macho] Use uint64_t instead of size_t to fix ...

David Spickett via All-commits all-commits at lists.llvm.org
Thu Feb 16 01:45:15 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0e1fb48bb95abaf94c0d21b58c4c075f6faad8ba
      https://github.com/llvm/llvm-project/commit/0e1fb48bb95abaf94c0d21b58c4c075f6faad8ba
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2023-02-16 (Thu, 16 Feb 2023)

  Changed paths:
    M lld/MachO/ConcatOutputSection.cpp

  Log Message:
  -----------
  [lld-macho] Use uint64_t instead of size_t to fix 32 bit test failures

Our bot has been failing https://lab.llvm.org/buildbot/#/builders/178/builds/3967:
Assertion `isecEnd - isecVA <= forwardBranchRange && "should only finalize sections in jump range"' failed.

I think this is due to the use of size_t, which is 32 bit on 32 bit,
for a value used in some 64 bit address calculations. Which was added in
https://reviews.llvm.org/D144029.

Switching to uint64_t fixes the issues.




More information about the All-commits mailing list