[CodeGen] "PromoteInteger" f32 to f64 doesn't make sense.
authorAhmed Bougacha <ahmed.bougacha@gmail.com>
Sat, 28 Mar 2015 01:22:37 +0000 (01:22 +0000)
committerAhmed Bougacha <ahmed.bougacha@gmail.com>
Sat, 28 Mar 2015 01:22:37 +0000 (01:22 +0000)
commit44c1c890e459459fdd039f3e9c200ce35d1ebc54
treed2c7ef99ae5de86500f32eb270014448fc4d1a0b
parent57e9efecb08c15e29e63b2ff2f8f64ff511fd7ad
[CodeGen] "PromoteInteger" f32 to f64 doesn't make sense.

The original f32->f64 promotion logic was refactored into roughly the
currently shape in r37781. However, starting with r132263, the
legalizer has been split into different kinds, and the previous
"Promote" (which did the right thing) was search-and-replace'd into
"PromoteInteger". The divide gradually deepened, with type legalization
("PromoteInteger") being separated from ops legalization
("Promote", which still works for floating point ops).

Fast-forward to today: there's no in-tree target with legal f64 but
illegal f32 (rather: no tests were harmed in the making of this patch).

With such a target, i.e., if you trick the legalizer into going through
the PromoteInteger path for FP, you get the expected brokenness.
For instance, there's no PromoteIntRes_FADD (the name itself sounds
wrong), so we'll just hit some assert in the PromoteInteger path.

Don't pretend we can promote f32 to f64. Instead, always soften.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@233464 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/TargetLoweringBase.cpp