[PATCH] D30895: [ELF] - Step to combine LinkerScript and LinkerScriptBase

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 08:06:46 PDT 2017


grimar created this revision.

We can move all not templated functionality to LinkerScriptBase.
Patch do that for hasPhdrsCommands() and shows how it helps to detemplate
things in other places.

Probably we should be able to merge these 2 classes into single one after such steps.
Even if not, it still looks as reasonable cleanup for me.


https://reviews.llvm.org/D30895

Files:
  ELF/LinkerScript.cpp
  ELF/LinkerScript.h
  ELF/Writer.cpp
  ELF/Writer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30895.91564.patch
Type: text/x-patch
Size: 5530 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170313/6a58e050/attachment.bin>


More information about the llvm-commits mailing list