This is an update to a previous commit (r181216).
authorJean-Luc Duprat <jduprat@apple.com>
Wed, 22 May 2013 18:29:31 +0000 (18:29 +0000)
committerJean-Luc Duprat <jduprat@apple.com>
Wed, 22 May 2013 18:29:31 +0000 (18:29 +0000)
commit5e6cabd163c9c207df3778f3508b0de445578057
tree1c7d2157422f09ffd9ad2df66a07ec67f2162597
parente0f26bc008bf7eb176595ab170cca86e21a6eae8
This is an update to a previous commit (r181216).

The earlier change list introduced the following inst combines:
B * (uitofp i1 C) —> select C, B, 0
A * (1 - uitofp i1 C) —> select C, 0, A
select C, 0, B + select C, A, 0 —> select C, A, B

Together these 3 changes would simplify :
A * (1 - uitofp i1 C) + B * uitofp i1 C
down to :
select C, B, A

In practice we found that the first two substitutions can have a
negative effect on performance, because they reduce opportunities to
use FMA contractions; between the two options FMAs are often the
better choice.  This change list amends the previous one to enable
just these inst combines:

select C, B, 0 + select C, 0, A —> select C, B, A
A * (1 - uitofp i1 C) + B * uitofp i1 C —> select C, B, A

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@182499 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/InstCombine/InstCombineAddSub.cpp
lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
test/Transforms/InstCombine/add4.ll