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

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 14:54:38 PDT 2016


On 9 August 2016 at 14:26, Hans Wennborg <hans at chromium.org> wrote:
> Since someone ran into this with real code, perhaps we should consider
> to merge it?
>
> Tim, Renato: what do you think?

It seems like a strict improvement, though a bit oddly written (v4i64
is never legal on AArch64). I also suspect performFDivCombine is
similarly affected for the int -> float conversion.

Tim.


More information about the llvm-commits mailing list