[PATCH] D15730: [MachineLICM] Fix handling of memoperands

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 11:03:56 PST 2015


reames added a comment.

In http://reviews.llvm.org/D15730#316291, @atrick wrote:

> AArch64 uses a concatenateMemOperands utility.


Looking around, I see three copes of this routine (2 in tree, 1 under review).  I think it makes sense to common this functionality.  Would a static function on MachineInstr be a reasonable place for this?  Or is there a better location for this type of utility function.


Repository:
  rL LLVM

http://reviews.llvm.org/D15730





More information about the llvm-commits mailing list