[PATCH] D41277: Implement --just-symbols
Chih-Mao Chen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 15 20:24:13 PST 2017
PkmX marked 3 inline comments as done.
PkmX added inline comments.
================
Comment at: ELF/Driver.cpp:183
+ if (JustSymbols) {
+ Files.push_back(createJustSymbolsFile(MBRef));
----------------
ruiu wrote:
> You can just call `Files.push_back(createJustSymbolsFile(MBRef))` instead of calling `addFile`.
You are right. In retrospect I should've just call that instead of adding an extra argument to `addFile`.
================
Comment at: ELF/InputFiles.h:337
+ static bool classof(const InputFile *F) {
+ return F->kind() == InputFile::JustSymbolsKind;
+ }
----------------
ruiu wrote:
> Remove `InputFile::`.
The `InputFile::` is required due to the rules of two-phase lookup in C++.
https://reviews.llvm.org/D41277
More information about the llvm-commits
mailing list