[PATCH] D27323: [X86] Fix non-intrinsic roundss/roundsd to not read the destination register

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 17:04:10 PST 2016


craig.topper added a comment.

Should these rows in X86InstrInfo.cpp be moved to MemoryFoldTable1 from MemoryFoldTable2?

  { X86::ROUNDSDr,        X86::ROUNDSDm,      0 },
  { X86::ROUNDSSr,        X86::ROUNDSSm,      0 },

And should ROUNDSSr_Int and ROUNDSDr_Int be added to MemoryFoldTable2? And related to that should RSQRTSSr_Int, RCPSSr_Int, SQRTSDr_Int, and SQRTSSr_Int actually be in MemoryFoldTable2 instead of MemoryFoldTable1?


https://reviews.llvm.org/D27323





More information about the llvm-commits mailing list