[PATCH] D74485: [ARM] Fix ReconstructShuffle for bigendian

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 06:41:40 PST 2020


simon_tatham accepted this revision.
simon_tatham added a comment.
This revision is now accepted and ready to land.

LGTM, with a not-very-important comment nitpick.



================
Comment at: llvm/lib/Target/ARM/ARMInstrInfo.td:313
+// whereas 'bitconvert' will map it to the high byte in big-endian mode,
+// because that's what VSTRH.16 followed by VLDRB.8 would do. So the bitconvert
+// would have to emit a VREV16.8 instruction, whereas the VECTOR_REG_CAST emits
----------------
This reference to MVE-specific instruction names might be out of place now this comment is shared with NEON :-) But I don't know enough NEON to be sure of what the analogous load/store instructions there look like. Are those VST1 / VLD1, perhaps?


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

https://reviews.llvm.org/D74485





More information about the llvm-commits mailing list