[PATCH] D94990: [ARM] Simplify extract of VMOVDRR

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 27 08:23:20 PST 2021


SjoerdMeijer added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:14097
+    if (MOV.getOpcode() == ARMISD::VMOVDRR)
+      return MOV.getOperand(ST->isLittle() ? Offset % 2 : 1 - Offset % 2);
+  }
----------------
Sorry for the kind of usual test nag question, but I see `isLittle` here and was wondering if we have or need tests for BE?


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

https://reviews.llvm.org/D94990



More information about the llvm-commits mailing list