[PATCH] D45375: [ELF] - Introduce synthetic file for linker script symbols.
Rafael Avila de Espindola via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 10 16:21:29 PDT 2018
espindola added a comment.
Please rebase.
================
Comment at: ELF/InputFiles.h:73
+
+ static InputFile* createSyntheticFile(StringRef Name);
+
----------------
clang-format
================
Comment at: ELF/LTO.cpp:162
// will be generated by for them, triggering linker errors.
- // Symbol section is always null for bitcode symbols, hence the check
- // for isElf(). Skip linker script defined symbols as well: they have
----------------
You should keep the comment about the section symbol being null for bitcode symbols, that is why we need the isa<BitcodeFile>
https://reviews.llvm.org/D45375
More information about the llvm-commits
mailing list