[PATCH] D37467: Add a new pass to speculate around PHI nodes with constant (integer) operands when profitable.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc updated this revision to Diff 119279.
chandlerc added a comment.

Add a stricter form of checking for profitability to avoid one potential issue
with this patch. Also clean up comments in various places to try and help
address code review feedback.


https://reviews.llvm.org/D37467

Files:
  llvm/include/llvm/Transforms/Scalar/SpeculateAroundPHIs.h
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Passes/PassRegistry.def
  llvm/lib/Transforms/Scalar/CMakeLists.txt
  llvm/lib/Transforms/Scalar/SpeculateAroundPHIs.cpp
  llvm/test/Other/new-pm-defaults.ll
  llvm/test/Other/new-pm-thinlto-defaults.ll
  llvm/test/Transforms/SpeculateAroundPHIs/X86/basic.ll
  llvm/test/Transforms/SpeculateAroundPHIs/X86/lit.local.cfg

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37467.119279.patch
Type: text/x-patch
Size: 58623 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171017/e9f67487/attachment.bin>