[PATCH] D40777: [dsymutil] Add -verify option to run DWARF verifier after linking.

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 4 17:38:34 PST 2017


probinson added inline comments.


================
Comment at: tools/dsymutil/dsymutil.cpp:105
+         "default."),
+    value_desc("arch"), ZeroOrMore, cat(DsymCategory));
 
----------------
This looks like an unrelated formatting change?
Also later on a couple of extra blank lines were removed.  I'm guessing you ran clang-format on the whole file instead of using clang-format-diff.  Generally we prefer whitespace/formatting changes to be done separately from functional changes.


Repository:
  rL LLVM

https://reviews.llvm.org/D40777





More information about the llvm-commits mailing list