[PATCH] D15757: Extract helper function to merge MemoryOperand lists [NFC]

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 12:39:55 PST 2015


reames created this revision.
reames added reviewers: atrick, sanjoy, qcolombet, t.p.northover.
reames added a subscriber: llvm-commits.

In the discussion on http://reviews.llvm.org/D15730, Andy pointed out we had a utility function for merging MMO lists.  Since it turned we actually had two copies and there's another review in progress (http://reviews.llvm.org/D15230) which needs the same, extract it into a utility function and clean up the interfaces to make it easier to use with a MachineInstBuilder.  

I introduced a pair here to track size and allocation together.  I think we should probably move in the direction of the MachineOperandsRef helper class, but I'm leaving that for further work.  I want to get the poison state introduced before I make major changes to the interface.  

http://reviews.llvm.org/D15757

Files:
  include/llvm/CodeGen/MachineInstr.h
  include/llvm/CodeGen/MachineInstrBuilder.h
  lib/CodeGen/MachineInstr.cpp
  lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
  lib/Target/ARM/ARMLoadStoreOptimizer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15757.43555.patch
Type: text/x-patch
Size: 7742 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151223/21571b08/attachment.bin>


More information about the llvm-commits mailing list