[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