This is an archive of the discontinued LLVM Phabricator instance.

[X86] Remove X86ISD::FMADD in favor ISD::FMA
ClosedPublic

Authored by craig.topper on Aug 21 2017, 3:04 PM.

Details

Summary

There's no reason to have a target specific node with the same semantics as a target independent opcode.

This should simplify D36335 so that it doesn't need to touch X86ISelDAGToDAG.cpp

Diff Detail

Event Timeline

craig.topper created this revision.Aug 21 2017, 3:04 PM
zvi accepted this revision.Aug 23 2017, 9:12 AM

LGTM

This revision is now accepted and ready to land.Aug 23 2017, 9:12 AM
This revision was automatically updated to reflect the committed changes.