[PATCH] D26521: [X86] Allow folding of reloads from stack slots when loading a subreg of the spilled reg

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 14 13:50:32 PST 2016


mkuper updated this revision to Diff 77878.
mkuper added a comment.

Updated per comments, and rebased to show test diffs.

Also moved the code that computes MemSize to be before calling into the target's foldMemoryOperandImpl() because, apparently, the X86 implementation feels free to change MI, so accessing MI.getOperand(Idx) after the Impl call may produce unexpected results.


https://reviews.llvm.org/D26521

Files:
  include/llvm/Target/TargetInstrInfo.h
  lib/CodeGen/InlineSpiller.cpp
  lib/CodeGen/TargetInstrInfo.cpp
  lib/Target/X86/X86InstrInfo.cpp
  lib/Target/X86/X86InstrInfo.h
  test/CodeGen/X86/partial-fold32.ll
  test/CodeGen/X86/partial-fold64.ll
  test/CodeGen/X86/vector-half-conversions.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26521.77878.patch
Type: text/x-patch
Size: 10820 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161114/e9e219c1/attachment.bin>


More information about the llvm-commits mailing list