[PATCH] D25991: [PPC] Peephole to remove extra fcmp that checks for NaN

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 18:12:58 PDT 2016

echristo added inline comments.

Comment at: lib/Target/PowerPC/PPCInstrInfo.cpp:1543
+                                      const MachineRegisterInfo *MRI) const {
+  // We want to remove the first fcmpu in patterns like this
+  //      fcmpu 0, 1, 1
Actually, go ahead and put this in the commit message as well. That way there's an easier record of it there too. More detailed commit messages are often goodness.


More information about the llvm-commits mailing list