[PATCH] D53006: [llvm-objcopy] Factor out CopyConfig

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 9 10:45:13 PDT 2018


rupprecht added a comment.

It's nice to see the size of llvm-objcopy.cpp cut in half :)
LGTM, but I'll let one of the others give the approval stamp.



================
Comment at: tools/llvm-objcopy/CopyConfig.h:24
+namespace llvm {
+enum class DebugCompressionType;
+
----------------
Doesn't the #include for TargetOptions.h provide this?


Repository:
  rL LLVM

https://reviews.llvm.org/D53006





More information about the llvm-commits mailing list