[PATCH] D58470: [X86] Reduce some patterns by using FP instructions for integer types even when AVX2 is available and execution domain fixing will do the right thing

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 04:07:22 PST 2019


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - as long as you are happy with the potential -O0 domain changes.

In my experience, load folding seems to get most things right but there are some cases (MOVSD vs BLENDPD etc.) where we struggle to recover from.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58470/new/

https://reviews.llvm.org/D58470





More information about the llvm-commits mailing list