[PATCH] D15429: Fix bug with FMA Mutate and register classes

Kyle Butt via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 12:54:46 PST 2015


iteratee marked 4 inline comments as done.

================
Comment at: lib/Target/PowerPC/PPCVSXFMAMutate.cpp:270
@@ -264,4 +269,3 @@
 
-          UseMO.setReg(KilledProdReg);
-          UseMO.setSubReg(KilledProdSubReg);
+          UseMO.substVirtReg(KilledProdReg, KilledProdSubReg, *TRI);
         }
----------------
hfinkel wrote:
> Is this a drive-by cleanup or part of the fix? If the former, please commit it separately.
It's part of the fix. I actually fixed this first, and it lead to using VR registers that didn't exist, so I had to add the constraint.


Repository:
  rL LLVM

http://reviews.llvm.org/D15429





More information about the llvm-commits mailing list