[PATCH] D22564: [ELF] Support FLAGS attribute in PHDR definition
Eugene Leviant via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 20 07:11:40 PDT 2016
evgeny777 added inline comments.
================
Comment at: ELF/LinkerScript.cpp:620
@@ -618,3 +619,3 @@
StringRef Tok = next();
- Opt.PhdrsCommands.push_back({Tok, PT_NULL, false, false});
+ Opt.PhdrsCommands.push_back({Tok, PT_NULL, false, false, false, 0});
PhdrsCommand &PhdrCmd = Opt.PhdrsCommands.back();
----------------
ruiu wrote:
> Set Flags to PF_R by default. Then you should be able to remove HasFlags member.
I have small concern about such change. Currently if we don't set PHDR flags in script then flags are OR-combined from member section flags. If we set flags in script then I guess we can't change them while assigning sections to segment, can we?
Repository:
rL LLVM
https://reviews.llvm.org/D22564
More information about the llvm-commits
mailing list