[X86] Simplify the BRCOND handling for FCMP_UNE.
authorCraig Topper <craig.topper@intel.com>
Thu, 10 Jan 2019 19:02:14 +0000 (19:02 +0000)
committerCraig Topper <craig.topper@intel.com>
Thu, 10 Jan 2019 19:02:14 +0000 (19:02 +0000)
commit350e6e9d7c84d30a04de3adb7a156afcc4c8a353
tree031438890c6d6a56973c7987bc202e1ded58949b
parenta936f848634ee18ae1ebffcc57b79f05f292cb17
[X86] Simplify the BRCOND handling for FCMP_UNE.

Despite what the comment says, FCMP_UNE would be an OR not an AND. In the lowering code the first branch created still goes to the original destination. The second branch was exchanged to go to where the subsequent unconditional branch went. This is different than what we do for FCMP_OEQ where both branches that we create go to the original unconditional branch.

As far as I can tell, I think this means we don't need to exchange the branch target with the unconditional branch for FCMP_UNE at all.

Differential Revision: https://reviews.llvm.org/D56309

llvm-svn: 350873
llvm/lib/Target/X86/X86ISelLowering.cpp