[llvm-commits] [PATCH] PR970: isFloatingPoint audit 5 of 5

2007-01-20 Thread Gordon Henriksen
This final patch simplifies the logic in SelectionDAGISel.cpp with no functional change. This construct was in use:    if (I.getType()-isFloatingPoint())      visitFPBinary(I, ISD::FADD, ISD::VADD);     else      visitIntBinary(I, ISD::ADD, ISD::VADD); Which confusingly sent even floating-point

Re: [llvm-commits] [PATCH] PR970: isFloatingPoint audit 5 of 5

2007-01-20 Thread Reid Spencer
Hi Gordon, Thanks for the patches. I've committed all 5 in your patch series. There were only minor issues which I corrected: 1. I just remove m_Neg pattern matcher since you commented it out and it appears to be dead in llvm. 2. Your changes to CmpInst::CmpInst were not