[PATCH] D35960: [MachineOutliner] Cleanup: move findCandidates out of suffix tree

[Jessica Paquette via Phabricator via llvm-commits]

paquette created this revision.

Doing some cleanup on the outliner in preparation for some functional changes. This patch moves findCandidates out of the suffix tree. (No test attached since it's just some tidying that doesn't impact the behaviour of the outliner.) Also adds a couple FIXMEs for future cleanup/improvements.

It's much easier to see what's going on when benefit calculations and queries are in the same place. Because the suffix tree ought not to know target-specific details, it makes sense to yank this out of the tree.


https://reviews.llvm.org/D35960

Files:
  lib/CodeGen/MachineOutliner.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35960.108532.patch
Type: text/x-patch
Size: 23631 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170727/77fcfe5c/attachment.bin>