]> granicus.if.org Git - llvm/commit
[X86] Fix mistake that marked VADDSSrrb_Int/VADDSDrrb_Int/VMULSSrrb_Int/VMULSDrrb_Int...
authorCraig Topper <craig.topper@intel.com>
Wed, 5 Jun 2019 21:00:31 +0000 (21:00 +0000)
committerCraig Topper <craig.topper@intel.com>
Wed, 5 Jun 2019 21:00:31 +0000 (21:00 +0000)
commit996a7da5f7f98bc4a2cf276d829475fa3af4aed4
treef3edcd8a6f91db23ab754b065fc4f904d85a75ca
parent0a65caa64f7bde5183a5d36098be9bbced8a9dfd
[X86] Fix mistake that marked VADDSSrrb_Int/VADDSDrrb_Int/VMULSSrrb_Int/VMULSDrrb_Int as commutable.

One of the sources controls the pass through value for the upper bits
of the result so we can't really commute it.

In practice this problem isn't a functional issue because we would
only try to commute this instruction in order to fold a load. But
we can't do embedded rounding and fold a load at the same time. So
the load fold would never succeed so I don't think we would ever
commute or at least keep the version after commuting.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@362647 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/X86/X86InstrAVX512.td