[all-commits] [llvm/llvm-project] 5a4a05: [ARM] Add additional fmin/fmax with nan tests (NFC)

[Nikita Popov via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5a4a05c8116ebdcb434cd15796a255cf024a6bf0
      https://github.com/llvm/llvm-project/commit/5a4a05c8116ebdcb434cd15796a255cf024a6bf0
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-09-09 (Wed, 09 Sep 2020)

  Changed paths:
    A llvm/test/CodeGen/ARM/fminmax-folds.ll

  Log Message:
  -----------
  [ARM] Add additional fmin/fmax with nan tests (NFC)

Adding these to ARM which has both FMINNUM and FMINIMUM.


  Commit: 0a5dc7effb191eff740e0e7ae7bd8e1f6bdb3ad9
      https://github.com/llvm/llvm-project/commit/0a5dc7effb191eff740e0e7ae7bd8e1f6bdb3ad9
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-09-09 (Wed, 09 Sep 2020)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/ARM/fminmax-folds.ll
    M llvm/test/CodeGen/X86/fmaxnum.ll
    M llvm/test/CodeGen/X86/fminnum.ll

  Log Message:
  -----------
  [DAGCombiner] Fold fmin/fmax of NaN

fminnum(X, NaN) is X, fminimum(X, NaN) is NaN. This mirrors the
behavior of existing InstSimplify folds.

This is expected to improve the reduction lowerings in D87391,
which use NaN as a neutral element.

Differential Revision: https://reviews.llvm.org/D87415


Compare: https://github.com/llvm/llvm-project/compare/2955a27abc25...0a5dc7effb19