[PATCH] D37121: [DivRemHoist] add a pass to move div/rem pairs into the same block (PR31028)

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 112732.
spatel marked 2 inline comments as done.
spatel added a comment.

Patch updated:

1. Check dominance of the basic blocks rather than the instructions to avoid potential unexpected complexity.
2. Move the hoisted instruction after (rather than before) the other intruction using removeFromParent()+insertAfter().


https://reviews.llvm.org/D37121

Files:
  include/llvm/InitializePasses.h
  include/llvm/Transforms/Scalar.h
  include/llvm/Transforms/Scalar/DivRemHoist.h
  lib/Passes/PassBuilder.cpp
  lib/Passes/PassRegistry.def
  lib/Transforms/IPO/PassManagerBuilder.cpp
  lib/Transforms/Scalar/CMakeLists.txt
  lib/Transforms/Scalar/DivRemHoist.cpp
  lib/Transforms/Scalar/Scalar.cpp
  test/Transforms/DivRemHoist/div-rem-pairs.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37121.112732.patch
Type: text/x-patch
Size: 18675 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170825/9796ebfe/attachment.bin>