[PATCH] D59982: [ELF] Rename SyntheticSection::empty to more appropriate needed() with opposite meaning

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 29 01:57:48 PDT 2019


grimar added a comment.

I think renaming makes sense.
I also can imagine a different wording in this renaming: `empty()` -> `used()`.
Not sure what is better though.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D59982





More information about the llvm-commits mailing list