[PATCH] D58812: [ARM] Consider undefined-on-NaN conditions in checkVSELConstraints

[Oliver Stannard via Phabricator via llvm-commits]

olista01 created this revision.
olista01 added reviewers: efriedma, SjoerdMeijer.
Herald added subscribers: kristof.beyls, javed.absar.
Herald added a project: LLVM.

This function was not checking for the condition code variants which are undefined if either input is NaN, so we were missing selection of the VSEL instruction in some cases when using -fno-honor-nans or -ffast-math.


Repository:
  rL LLVM

https://reviews.llvm.org/D58812

Files:
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/vsel.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58812.188862.patch
Type: text/x-patch
Size: 19850 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190301/3f357bce/attachment.bin>