Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

[ELF] - Implement INSERT BEFORE.
ClosedPublic
Actions

Authored by grimar on Mar 12 2018, 6:38 AM.

Details

Reviewers

ruiu
• espindola

Commits

rG796684b451b4: [ELF] - Implement INSERT BEFORE.
rLLD327378: [ELF] - Implement INSERT BEFORE.
rL327378: [ELF] - Implement INSERT BEFORE.

Summary

This finishes PR35877.

INSERT BEFORE used similar to INSERT AFTER,
it inserts sections before the given target section.

Diff Detail

Repository: rL LLVM

Event Timeline

grimar created this revision.Mar 12 2018, 6:38 AM

Herald added subscribers: arichardson, emaste. · View Herald TranscriptMar 12 2018, 6:38 AM

George Rimar via Phabricator via llvm-commits
<llvm-commits@lists.llvm.org> writes:

for (std::pair<StringRef, std::vector<BaseCommand *>> &P :

InsertAfterCommands)

if (!P.second.empty())

error("unable to INSERT AFTER " + P.first + ": section not defined");

+
+ auto Check = [](llvm::DenseMap<StringRef, std::vector<BaseCommand *>> &Map) {
+ for (std::pair<StringRef, std::vector<BaseCommand *>> &P : Map)
+ if (!P.second.empty())
+ error("unable to INSERT AFTER/BEFORE " + P.first +
+ ": section not defined");
+ };
+ Check(InsertBeforeCommands);
+ Check(InsertAfterCommands);

It might be simpler to do

for (auto &Cmds : {InsertBeforeCommands, InsertAfterCommands}) {