[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