diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2020-04-12 09:47:03 -0400 |
|---|---|---|
| committer | Matt Arsenault <Matthew.Arsenault@amd.com> | 2020-04-13 10:25:39 -0700 |
| commit | e6605a209cc7785ae635fee1df7ff4c136d35c17 (patch) | |
| tree | 80509a8bbe2a135e7e9a7cee6b38f351af96760f /llvm/lib/CodeGen/MachineBasicBlock.cpp | |
| parent | 6dbf1a1229ba65f65784af1fd9ca2a780f0dc8b2 (diff) | |
| download | llvm-e6605a209cc7785ae635fee1df7ff4c136d35c17.zip llvm-e6605a209cc7785ae635fee1df7ff4c136d35c17.tar.gz llvm-e6605a209cc7785ae635fee1df7ff4c136d35c17.tar.bz2 | |
DAG: Fix wrong legality check for ISD::FMAD
Since 1725f2884175ca618d29b06e35f5c6ebd618053d, this should check
isFMADLegalForFAddFSub rather than the the plain isOperationLegal.
This would assert in a subset of cases due to an oddity in how FMAD is
selected. We will allow FMA formation pre-legalize, but not FMAD even
in cases where it would be valid.
The current hook requires passing in the root fadd/fsub. However, in
this distributed case, this would be far more complicated to pass in
the relevant operand. AMDGPU doesn't get any value from the node, and
only needs the type and is the only implementor, so I'm not sure why
we have this complexity. Just rename and expand the assert to avoid
the more complicated checks spread through the distribution logic.
Diffstat (limited to 'llvm/lib/CodeGen/MachineBasicBlock.cpp')
0 files changed, 0 insertions, 0 deletions