[PATCH] D68062: Propeller lld framework for basicblock sections

[Han Shen via Phabricator via llvm-commits]

shenhan updated this revision to Diff 228939.
shenhan added a comment.
Herald added a subscriber: mgorny.

Hi, we've refactored propeller and cleaned up dependencies and interfaces. Here are the change summaries:

1. all propeller files are moved into lld/ELF/Propeller,  and lld/ELF/Propeller/* no longer depend on any lld/ELF header files (but they can depend on lld/include/lld/**)

2. except 1 file - lld/ELF/LinkerPropeller.h&cpp, which defines interface between lld and propeller (and it can access lld/ELF/*.h files), all current and future interactions between these two must be defined here.

3. lld/ELF/Driver.cpp now only has 1 call into propeller. (and other flag processing things.)

Please take another look, thanks.


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68062/new/

https://reviews.llvm.org/D68062

Files:
  lld/ELF/Driver.cpp
  lld/ELF/LinkerPropeller.cpp
  lld/ELF/LinkerPropeller.h
  lld/ELF/Propeller/CMakeLists.txt
  lld/ELF/Propeller/Propeller.cpp
  lld/ELF/Propeller/Propeller.h
  lld/ELF/Propeller/PropellerCFG.cpp
  lld/ELF/Propeller/PropellerCFG.h
  lld/ELF/Propeller/PropellerConfig.h
  lld/ELF/Propeller/propeller_cfg.proto
  lld/include/lld/Common/PropellerCommon.h
  lld/test/ELF/propeller/Inputs/bad-propeller-1.data
  lld/test/ELF/propeller/Inputs/bad-propeller-2.data
  lld/test/ELF/propeller/Inputs/bad-propeller-3.data
  lld/test/ELF/propeller/Inputs/bad-propeller-4.data
  lld/test/ELF/propeller/Inputs/bad-propeller-5.data
  lld/test/ELF/propeller/Inputs/propeller-2.data
  lld/test/ELF/propeller/Inputs/propeller-3.data
  lld/test/ELF/propeller/Inputs/propeller.data
  lld/test/ELF/propeller/Inputs/sample.c
  lld/test/ELF/propeller/propeller-bad-profile-1.s
  lld/test/ELF/propeller/propeller-bad-profile-2.s
  lld/test/ELF/propeller/propeller-bad-profile-3.s
  lld/test/ELF/propeller/propeller-bad-profile-4.s
  lld/test/ELF/propeller/propeller-bad-profile-5.s
  lld/test/ELF/propeller/propeller-bbsections-dump.s
  lld/test/ELF/propeller/propeller-compressed-strtab-lto.s
  lld/test/ELF/propeller/propeller-compressed-strtab.s
  lld/test/ELF/propeller/propeller-error-on-bblabels.s
  lld/test/ELF/propeller/propeller-keep-bb-symbols.s
  lld/test/ELF/propeller/propeller-lto-bbsections-dump.s
  lld/test/ELF/propeller/propeller-opt-all-combinations.s
  lld/test/ELF/propeller/propeller-skip.s
  lld/test/ELF/propeller/propeller-symbol-order-dump.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68062.228939.patch
Type: text/x-patch
Size: 110356 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191112/89c21927/attachment.bin>