[PATCH] D44098: [ARM] Relax condition for PerformSHLSimplify

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 5 08:57:06 PST 2018


samparker created this revision.
samparker added a reviewer: john.brawn.
Herald added subscribers: kristof.beyls, javed.absar.

PerformSHLSimplify previously would be bail if one of given node's users had a shl operand because the transformation wouldn't necessarily combine the shift into the user nodes. I guess this should have also happened for other shifts too... Anyway, by removing this restriction we can still remove mov instructions or a mov is replaced by a shift, which sometimes can help register pressure.


https://reviews.llvm.org/D44098

Files:
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/load-combine-big-endian.ll
  test/CodeGen/ARM/load-combine.ll
  test/CodeGen/ARM/unfold-shifts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44098.137014.patch
Type: text/x-patch
Size: 13645 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180305/1d10324d/attachment.bin>


More information about the llvm-commits mailing list