Merging r278002:
authorHans Wennborg <hans@hanshq.net>
Wed, 10 Aug 2016 16:27:26 +0000 (16:27 +0000)
committerHans Wennborg <hans@hanshq.net>
Wed, 10 Aug 2016 16:27:26 +0000 (16:27 +0000)
commita3332eebf6971d48c0ad16402d258c3773dfe2cf
tree5255ce50ab60fc506a3d8b51de63ddbd89a66076
parent01cffbe7f86bdb5d7a3f52e8bea94ae24ba70f6a
Merging r278002:
------------------------------------------------------------------------
r278002 | sbaranga | 2016-08-08 06:13:57 -0700 (Mon, 08 Aug 2016) | 18 lines

[AArch64] PR28877: Don't assume we're running after legalization when creating vcvtfp2fxs

Summary:
The DAG combine transformation that was generating the
aarch64_neon_vcvtfp2fxs node was assuming that all
inputs where legal and wasn't accounting that the input
could be a v4f64 if we're trying to do the transformation
before legalization. We now bail out in this case.

All illegal types besides v4f64 were already rejected.

Fixes https://llvm.org/bugs/show_bug.cgi?id=28877.

Reviewers: jmolloy

Subscribers: aemerson, rengolin, llvm-commits

Differential Revision: https://reviews.llvm.org/D23261
------------------------------------------------------------------------

git-svn-id: https://llvm.org/svn/llvm-project/llvm/branches/release_39@278239 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/AArch64/AArch64ISelLowering.cpp
test/CodeGen/AArch64/aarch64-vcvtfp2fxs-combine.ll [new file with mode: 0644]