[PATCH] D49280: [X86] Remove isel patterns for MOVSS/MOVSD ISD opcodes with integer types.

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 13 14:24:52 PDT 2018


greened added a comment.

Ok, thanks for the explanation, that helps.  It sounds like no additional domain switching issues have been introduced, so this seems fine to me as far as that goes.

The extra moves are a little concerning.  Even with move elimination they still consume processor resources.  I'm not sure that trade-off is worth it just
to eliminate two really simple TableGen patterns.


Repository:
  rL LLVM

https://reviews.llvm.org/D49280





More information about the llvm-commits mailing list