[PATCH] D47414: [llvm-objcopy] Fix null symbol handling
Alexander Shaposhnikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 29 03:34:27 PDT 2018
alexshap added a comment.
in general i agree with @jhenderson. Regarding changing the definition of empty() or checking the size() (what i suggested) - the latter seemes to be more explicit to me and more intuitive / causing less surprise, but i don't insist.
Repository:
rL LLVM
https://reviews.llvm.org/D47414
More information about the llvm-commits
mailing list