[PATCH] D16829: An implementation of Swing Modulo Scheduling

[Brendon Cahoon via llvm-commits]

bcahoon updated this revision to Diff 60760.
bcahoon added a comment.

Hi Matthias - thank you for the review and the comments. I really appreciate it.  I think I've addressed all of your comments, unless I've inadvertently missed something.  The new patch includes a lot of changes based upon your comments.  The code has also been rebased.

Here's a summary of the changes:
I decided to use the name Pipeliner to be more consistent with naming, except for the test names which still start with swp.  I don't really have a strong preference of Pipeliner vs. SWP vs. SMS in the naming scheme.

I've moved the code to add the pipeliner pass to HexagonTargetMachine.cpp in the addPreRegAlloc method instead of doing it in TargetPassConfig.cpp.

I've added some more high-level comments at the beginning with references to a couple of useful papers.

I've converted many loops to be range-based for loops.

I've removed the comments from the method declarations so that they are not repeated in the method implementation.

I've removed unnecessary metadata, etc. from the tests.

Thanks,
Brendon


http://reviews.llvm.org/D16829

Files:
  include/llvm/CodeGen/Passes.h
  include/llvm/InitializePasses.h
  include/llvm/Target/TargetInstrInfo.h
  lib/CodeGen/CMakeLists.txt
  lib/CodeGen/CodeGen.cpp
  lib/CodeGen/MachinePipeliner.cpp
  lib/Target/Hexagon/HexagonInstrInfo.cpp
  lib/Target/Hexagon/HexagonInstrInfo.h
  lib/Target/Hexagon/HexagonTargetMachine.cpp
  test/CodeGen/Hexagon/swp-const-tc.ll
  test/CodeGen/Hexagon/swp-dag-phi.ll
  test/CodeGen/Hexagon/swp-epilog-reuse.ll
  test/CodeGen/Hexagon/swp-matmul-bitext.ll
  test/CodeGen/Hexagon/swp-max.ll
  test/CodeGen/Hexagon/swp-multi-loops.ll
  test/CodeGen/Hexagon/swp-vect-dotprod.ll
  test/CodeGen/Hexagon/swp-vmult.ll
  test/CodeGen/Hexagon/swp-vsum.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16829.60760.patch
Type: text/x-patch
Size: 184707 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160614/e00300e5/attachment-0001.bin>