[PATCH] D132036: [llvm-objdump] Add -dyld_info to llvm-otool

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 27 10:35:10 PDT 2022


thakis added inline comments.


================
Comment at: llvm/tools/llvm-objdump/MachODump.cpp:1362
+    consumeError(std::move(Err));
+    Err = Error::success();
+  }
----------------
nit: Not sure if this line is necessary; Error's move ctor comment sounds a bit like it already does this to the moved-from value: http://llvm-cs.pcc.me.uk/include/llvm/Support/Error.h#190


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

https://reviews.llvm.org/D132036



More information about the llvm-commits mailing list