[PATCH] D27705: [ARM] GlobalISel: Support i8/i16 ABI extensions

[Tim Northover via Phabricator via llvm-commits]

t.p.northover added inline comments.


================
Comment at: lib/Target/ARM/ARMInstructionSelector.cpp:78
+  if (Opc == G_SEXT)
+    return Size == 8 ? ARM::SXTB : ARM::SXTH;
+
----------------
If we don't already, it would be a very good idea to make sure we bail completely in Thumb mode. Incidentally, is there any particular reason you decided to go for ARM first? I thought most things were compiled in Thumb these days.


https://reviews.llvm.org/D27705