[PATCH] Refactor LowerFABS and LowerFNEG into one function (x86)
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6010 bytes
Desc: not available
More information about the llvm-commits