aganea added a comment. This is really a great change, thanks @beanz!
================ Comment at: llvm/tools/llvm-objcopy/llvm-objcopy.cpp:404 -int main(int argc, char **argv) { +int llvm_objcopy_main(int argc, char **argv) { InitLLVM X(argc, argv); ---------------- Shouldn't we say: ``` int objcopy_main(int argc, char **argv) { ``` here and the other places + all supporting code, if we want `llvm objcopy` (without the dash) like @phosek suggests? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109977/new/ https://reviews.llvm.org/D109977 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits