[PATCH] Refactor LowerFABS and LowerFNEG into one function (x86)

Sanjay Patel spatel at rotateright.com
Tue Aug 26 07:55:57 PDT 2014


Hi majnemer, chandlerc, nadav,

We duplicate ~30 lines of code to lower FABS and FNEG for x86, so I've combined them into one function. No functional change intended, so no additional test cases. Test-suite behavior is unchanged.

This is a follow-on patch to http://reviews.llvm.org/D5052 (use an integer constant for FABS / FNEG (x86)). 

If this patch is OK, we can abandon the other patch because that change is included here.

http://reviews.llvm.org/D5064

Files:
  lib/Target/X86/X86ISelLowering.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5064.12947.patch
Type: text/x-patch
Size: 6010 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140826/e591dd5f/attachment.bin>


More information about the llvm-commits mailing list