ruiu added inline comments. ================ Comment at: ELF/InputFiles.h:310 @@ +309,3 @@ +private: + std::unique_ptr<char[]> ELFData; +}; ---------------- Is this equivalent to `std::unique_ptr<char *>`? If so, please write that way. Can you use uint8_t instead of char? https://reviews.llvm.org/D24060