jmolloy accepted this revision. jmolloy added a comment. This revision is now accepted and ready to land. That'll be because it's clearing the MSB of the floating point value in the integer domain: fabs(float f) = ((int)x & ~(1U<<31)) LGTM. https://reviews.llvm.org/D24337