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

Ehsan Amiri via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 18:20:06 PDT 2016


amehsan 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
----------------
echristo wrote:
> 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.
Sure


https://reviews.llvm.org/D25991





More information about the llvm-commits mailing list