This is an archive of the discontinued LLVM Phabricator instance.

[x86] replace integer 'and' ops with packed SSE FP 'and' ops when operating on FP scalars
ClosedPublic

Authored by spatel on Sep 22 2015, 12:41 PM.

Details

Summary

As noted in the comment, we can extend this from just 'and' ops to the other logic ops if this is a good solution.

This is related to, but does not solve:
https://llvm.org/bugs/show_bug.cgi?id=22428

Diff Detail

Event Timeline

spatel updated this revision to Diff 35402.Sep 22 2015, 12:41 PM
spatel retitled this revision from to [x86] replace integer 'and' ops with packed SSE FP 'and' ops when operating on FP scalars.
spatel updated this object.
spatel added reviewers: qcolombet, chandlerc.
spatel added a subscriber: llvm-commits.
ab accepted this revision.Sep 22 2015, 2:35 PM
ab added a reviewer: ab.
ab added a subscriber: ab.

Sounds reasonable

lib/Target/X86/X86ISelLowering.cpp
24453

TODO -> FIXME ?

This revision is now accepted and ready to land.Sep 22 2015, 2:35 PM
spatel added inline comments.Sep 22 2015, 2:46 PM
lib/Target/X86/X86ISelLowering.cpp
24453

Sure - follow-up patch coming soon. :)
Thanks for the prompt review!

This revision was automatically updated to reflect the committed changes.