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


More information about the llvm-commits mailing list