[PATCH] D71743: [ARM][MVE] Enable masked gathers from vector of pointers

[Dave Green via Phabricator via llvm-commits]

dmgreen added a reviewer: samparker.
dmgreen added a comment.

Nice, thanks!



================
Comment at: llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp:1
+//===- ARMGatherScatterLowering.cpp - Gather/Scatter lowering ------------===//
+//
----------------
This looks a touch too short, unless my eyes deceive me.


================
Comment at: llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp:15
+
+#include <algorithm>
+#include <cassert>
----------------
It's customary in llvm to put std headers below the llvm headers.


================
Comment at: llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp:86
+
+bool isLegalAlignment(unsigned NumElements, unsigned ElemSize,
+                      unsigned Alignment) {
----------------
Any function that is only used in this file can be static.


================
Comment at: llvm/lib/Target/ARM/MVEGatherScatterLowering.cpp:178
+
+  bool ModifiedDT;
+  for (IntrinsicInst *I : Gathers)
----------------
This isn't used it seems.


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

https://reviews.llvm.org/D71743