[PATCH] use an integer constant for FABS / FNEG (x86)

Sanjay Patel spatel at rotateright.com
Mon Aug 25 12:22:36 PDT 2014


This patch is just a bit of cleanup to ease refactoring LowerFABS() and LowerFNEG() since they have a lot of overlap. 

I've removed the creation of a floating point constant from an integer because it's going to be used for a bitwise integer op anyway. 

I don't intend for this to make any difference to codegen, and AFAICT there isn't. There's no difference in test-suite either.

http://reviews.llvm.org/D5052

Files:
  lib/Target/X86/X86ISelLowering.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5052.12907.patch
Type: text/x-patch
Size: 1676 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140825/2321b9c1/attachment.bin>


More information about the llvm-commits mailing list