[AArch64] PR28877: Don't assume we're running after legalization when creating vcvtfp2fxs
authorSilviu Baranga <silviu.baranga@arm.com>
Mon, 8 Aug 2016 13:13:57 +0000 (13:13 +0000)
committerSilviu Baranga <silviu.baranga@arm.com>
Mon, 8 Aug 2016 13:13:57 +0000 (13:13 +0000)
commitfa00ba3c1a43b52779d9d23fd9baef57601f6ea1
tree235246bf654323f00681bbb0ffcabdd72cf51b54
parent4ec22ec6b19d341af8106db2dc1964729510558d
[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

llvm-svn: 278002
llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
llvm/test/CodeGen/AArch64/aarch64-vcvtfp2fxs-combine.ll [new file with mode: 0644]