[PATCH] D12093: [NVPTX] Support register copy from i16 to i32 register types

Justin Holewinski via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 11:51:01 PDT 2015


jholewinski added inline comments.

================
Comment at: lib/Target/NVPTX/NVPTXInstrInfo.cpp:39-40
@@ -38,4 +38,4 @@
   const TargetRegisterClass *SrcRC = MRI.getRegClass(SrcReg);
 
-  if (DestRC->getSize() != SrcRC->getSize())
-    report_fatal_error("Copy one register into another with a different width");
+  if (DestRC->getSize() != SrcRC->getSize()) {
+    // If the sizes differ it may be possible we are copying a i16 to a i32
----------------
Right, we don't define i8 registers, so a load of an i8 has to be put in an i16 register.  We still shouldn't have a cross-class register copy here, though.


http://reviews.llvm.org/D12093





More information about the llvm-commits mailing list