[PATCH] D105089: [llvm-objcopy][MachO] Code cleanup

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexander-shaposhnikov added inline comments.


================
Comment at: llvm/tools/llvm-objcopy/MachO/MachOReader.cpp:79
-      Sections.push_back(
-          std::make_unique<Section>(constructSection(*Curr, NextSectionIndex)));
-    }
----------------
@jhenderson this pointer dereference appears to be problematic.
Perhaps, we should drop reinterpret_cast on line 68 completely.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105089