[PATCH] D26872: Outliner: Add MIR-level outlining pass

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 12:57:54 PST 2017


paquette updated this revision to Diff 83675.
paquette marked 55 inline comments as done.
paquette added a comment.

Hi everyone,

I think I addressed most if not all of the comments from the previous patch; thanks for all of the help with reviewing this! I'm quite happy with how this has been coming along in progressing from intern project to real code.

Major changes:

- No use of raw new/delete in the outliner
- Tried to decouple the "program strings" from normal strings-- that seemed to be somewhat confusing
- New test which also checks if the outliner obeys basic block boundaries
- Lots of type changes; lots of things that didn't have to be pointers are no longer pointers
- More use of appropriate LLVM data structures for memory management, maps, etc.

Tell me what you think!


Repository:
  rL LLVM

https://reviews.llvm.org/D26872

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/MachineOutliner.cpp
  lib/CodeGen/TargetPassConfig.cpp
  lib/Target/X86/X86InstrInfo.cpp
  lib/Target/X86/X86InstrInfo.h
  test/CodeGen/X86/machineoutliner.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26872.83675.patch
Type: text/x-patch
Size: 60517 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170109/93ce858a/attachment-0001.bin>


More information about the llvm-commits mailing list