[PATCH] D18480: Reapply [LSR] Create fewer redundant instructions.

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 13:07:00 PDT 2016


gberry created this revision.
gberry added a reviewer: atrick.
gberry added subscribers: mcrosier, mzolotukhin, llvm-commits.

Fix LSRInstance::HoistInsertPosition() to check the original insert
position block first for a canonical insertion point that is dominated
by all inputs.  This leads to SCEV being able to reuse more instructions
since it currently tracks the instructions it creates for reuse by
keeping a table of <Value, insert point> pairs.

Originally reviewed in http://reviews.llvm.org/D18001

http://reviews.llvm.org/D18480

Files:
  lib/Transforms/Scalar/LoopStrengthReduce.cpp
  test/Transforms/LoopStrengthReduce/AArch64/lsr-reuse.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18480.51677.patch
Type: text/x-patch
Size: 3873 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160325/98ee7ed3/attachment.bin>


More information about the llvm-commits mailing list