[PATCH] D62967: [ARM] Add MVE addressing to isLegalT2AddressImmediate

[Sam Parker via Phabricator via llvm-commits]

samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:13305
+  if (VT.isVector()) {
+    if (Subtarget->hasMVEFloatOps()) {
+      switch (VT.getSimpleVT().getVectorElementType().SimpleTy) {
----------------
I think it would be cleaner to exit early for floats without support, like we do currently at the start of the function. Then we can have a single switch statement here.


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

https://reviews.llvm.org/D62967