[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