[PATCH] D123276: [lld-macho] Use fewer indirections in UnwindInfo implementation

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 11:27:25 PDT 2022


int3 added inline comments.


================
Comment at: lld/MachO/UnwindInfoSection.cpp:107
+struct CompactUnwindEntry {
+  uint64_t functionAddress;
+  uint32_t functionLength;
----------------
oontvoo wrote:
> IIRC, this was changed to `Ptr` (it was originally `uint64_t`) because of an overflow bug on 32-bit platforms ... 
> 
maybe you were thinking of {D107035}?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123276/new/

https://reviews.llvm.org/D123276



More information about the llvm-commits mailing list