[PATCH] D57122: Add copyWithSizeAndOffset to MMOs for splitting

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 23 15:23:57 PST 2019


paquette added inline comments.


================
Comment at: lib/CodeGen/GlobalISel/LegalizerHelper.cpp:501-502
-          MMO.getPointerInfo().getWithOffset(Adjustment), MMO.getFlags(),
-          NarrowSize / 8, Alignment, MMO.getAAInfo(), MMO.getRanges(),
-          MMO.getSyncScopeID(), MMO.getOrdering(), MMO.getFailureOrdering());
 
----------------
I assume that `MMO.getAAInfo()`, `MMO.getRanges()`, etc are automatically handled by the call to `getMachineMemOperand` in `copyWithSizeAndOffset`? Or are they no longer relevant?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57122/new/

https://reviews.llvm.org/D57122





More information about the llvm-commits mailing list