[InstCombine] fold bitcasts around an extractelement (2nd try)
authorSanjay Patel <spatel@rotateright.com>
Wed, 9 Dec 2015 18:57:16 +0000 (18:57 +0000)
committerSanjay Patel <spatel@rotateright.com>
Wed, 9 Dec 2015 18:57:16 +0000 (18:57 +0000)
commit1e3aaa8beff4a0797feb2ca2cb57668ab02a95fa
treeba47526ed6a382ddc4ceb830d87003cf6f763e9f
parent4c0fb60923bd45a297757929800a8ba97b531cbb
[InstCombine] fold bitcasts around an extractelement (2nd try)

This is a redo of r255124 (reverted at r255126) with an added check for a
scalar destination type and an added test for the failure seen in Clang's
test/CodeGen/vector.c. The extra test shows a different missing optimization.

Original commit message:

Example:
  bitcast (extractelement (bitcast <2 x float> %X to <2 x i32>), 1) to float
    --->
  extractelement <2 x float> %X, i32 1

This is part of fixing PR25543:
https://llvm.org/bugs/show_bug.cgi?id=25543

The next step will be to generalize this fold:
trunc ( lshr ( bitcast X) ) -> extractelement (X)

Ie, I'm hoping to replace the existing transform of:
bitcast ( trunc ( lshr ( bitcast X)))
added by:
http://reviews.llvm.org/rL112232

with 2 less specific transforms to catch the case in the bug report.

Differential Revision: http://reviews.llvm.org/D14879

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@255137 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/InstCombine/InstCombineCasts.cpp
test/Transforms/InstCombine/bitcast.ll