diff options
| author | Craig Topper <craig.topper@intel.com> | 2019-06-05 21:00:31 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2019-06-05 21:00:31 +0000 |
| commit | 3975b15dbabd9ee595bcbada2088dd1395f52e0f (patch) | |
| tree | cb32b5e6dacd9fd34bdc42bd0c56a97b25c3414a /llvm/lib | |
| parent | a3701caad826a6da3915a1b485cbe0a333e6a484 (diff) | |
| download | bcm5719-llvm-3975b15dbabd9ee595bcbada2088dd1395f52e0f.tar.gz bcm5719-llvm-3975b15dbabd9ee595bcbada2088dd1395f52e0f.zip | |
[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.
llvm-svn: 362647
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Target/X86/X86InstrAVX512.td | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/X86/X86InstrAVX512.td b/llvm/lib/Target/X86/X86InstrAVX512.td index b230cbf611b..4302b3e1ed5 100644 --- a/llvm/lib/Target/X86/X86InstrAVX512.td +++ b/llvm/lib/Target/X86/X86InstrAVX512.td @@ -5408,7 +5408,7 @@ multiclass avx512_fp_scalar_round<bits<8> opc, string OpcodeStr,X86VectorVTInfo (ins _.RC:$src1, _.RC:$src2, AVX512RC:$rc), OpcodeStr, "$rc, $src2, $src1", "$src1, $src2, $rc", (VecNode (_.VT _.RC:$src1), (_.VT _.RC:$src2), - (i32 timm:$rc)), IsCommutable>, + (i32 timm:$rc))>, EVEX_B, EVEX_RC, Sched<[sched]>; } multiclass avx512_fp_scalar_sae<bits<8> opc, string OpcodeStr,X86VectorVTInfo _, |
