[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