[PATCH] D95809: [lld-macho] Emit personalities in compact unwind

[Greg Clayton via Phabricator via llvm-commits]

clayborg added inline comments.


================
Comment at: lld/MachO/UnwindInfoSection.cpp:108
+// XXX: This assumes that all the sections containing function addresses have
+// already been finalized. Not sure this is valid assumption.
+static void relocateCompactUnwind(MergedOutputSection *compactUnwindSection,
----------------
Is there an assert or check you can put into the code to verify this assumption?


================
Comment at: lld/MachO/UnwindInfoSection.cpp:201
   cuPtrVector.reserve(cuCount);
-  for (const CompactUnwindEntry64 &cuEntry : cuVector)
+  for (CompactUnwindEntry64 &cuEntry : cuVector)
     cuPtrVector.emplace_back(&cuEntry);
----------------
We don't seem to modify the "cuEntry" in the loop below, can be put the "const" back and revert this change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95809