[PATCH] D15317: X86-FMA3: Defined ExeDomain for Scalar FMA3 opcodes

Vyacheslav Klochkov via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 7 16:28:25 PST 2015


v_klochkov created this revision.
v_klochkov added subscribers: llvm-commits, DavidKreitzer, RKSimon.

Please review the patch that defines the ExeDomain for scalar X86-FMA3 opcodes.

The problem was noticed by RKSimon (Simon Pilgrim) in http://reviews.llvm.org/D14762 ( D14762 ).
ExeDomain was defined for packed FMA opcodes, but not defined for scalar FMAs.

This trivial fix in X86InstrFMA.td caused the need in minor changes in 4 LIT tests.

(The same problem for scalar FMA4 opcodes was fixed by Simon Pilgrim just recently (4th of Dec 2015).)

Thank you,
Slava


http://reviews.llvm.org/D15317

Files:
  llvm/lib/Target/X86/X86InstrFMA.td
  llvm/test/CodeGen/X86/fma-commute-x86.ll
  llvm/test/CodeGen/X86/fma-intrinsics-x86.ll
  llvm/test/CodeGen/X86/fma-scalar-memfold.ll
  llvm/test/CodeGen/X86/fma_patterns.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15317.42126.patch
Type: text/x-patch
Size: 20688 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151208/a12944a7/attachment-0001.bin>


More information about the llvm-commits mailing list