[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