[PATCH] D53887: [HotColdSplitting] [WIP] Outline more than once per function

[Vedant Kumar via Phabricator via llvm-commits]

vsk updated this revision to Diff 172456.
vsk marked 5 inline comments as done.
vsk edited the summary of this revision.
vsk added a comment.

- While doing performance testing I found a miscompile in ./SingleSource/Regression/C++/EH/Regression-C++-class_hierarchy. I'll file a PR with more details by next week. It looks like it could be an existing bug that surfaces due to more aggressive outlining.
- Added tests (including the one from @sebpop's earlier patch).


https://reviews.llvm.org/D53887

Files:
  llvm/lib/Transforms/IPO/HotColdSplitting.cpp
  llvm/test/Transforms/HotColdSplit/eh-pads.ll
  llvm/test/Transforms/HotColdSplit/mark-the-whole-func-cold.ll
  llvm/test/Transforms/HotColdSplit/outline-disjoint-diamonds.ll
  llvm/test/Transforms/HotColdSplit/outline-multiple-entry-region.ll
  llvm/test/Transforms/HotColdSplit/region-overlap.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53887.172456.patch
Type: text/x-patch
Size: 27883 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181102/48725cf8/attachment.bin>