[PATCH] D15213: [SimplifyCFG] allow speculation of exactly one expensive instruction (PR24818)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 14:27:49 PST 2015


spatel updated this revision to Diff 42778.
spatel marked an inline comment as done.
spatel added a comment.

Patch updated based on James' feedback:

1. Fixed check to use !empty() rather than size().
2. Fixed check to specify 'Depth > 0' since it's an integer.
3. Added cl::opt override to disable this.


http://reviews.llvm.org/D15213

Files:
  lib/Transforms/Utils/SimplifyCFG.cpp
  test/Transforms/SimplifyCFG/X86/speculate-cttz-ctlz.ll
  test/Transforms/SimplifyCFG/speculate-math.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15213.42778.patch
Type: text/x-patch
Size: 6670 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151214/a60129d7/attachment.bin>


More information about the llvm-commits mailing list