[PATCH] D122703: [AArch64][InstCombine] Fold MLOAD and zero extensions into MLOAD

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm accepted this revision.
paulwalker-arm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:6056
 
-    // fold (and (masked_load) (build_vec (x, ...))) to zext_masked_load
     auto *MLoad = dyn_cast<MaskedLoadSDNode>(N0);
----------------
Sorry I think I wasn't clear enough with my previous comment.  I meant that it is not worth having both the original build_vec and your new splat_vec comment and so it would be better to just keep the splat_vec one (i.e. `fold (and (masked_load) (splat_vec (x, ...))) to zext_masked_load`)


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

https://reviews.llvm.org/D122703