[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.


https://reviews.llvm.org/D25991





More information about the llvm-commits mailing list