[all-commits] [llvm/llvm-project] 6cfd34: APFloat: Fix signed zero handling in minnum/maxnum...

Matt Arsenault via All-commits all-commits at lists.llvm.org
Thu Feb 29 03:23:23 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6cfd3439d4b99be85f647849168e1076a9737170
      https://github.com/llvm/llvm-project/commit/6cfd3439d4b99be85f647849168e1076a9737170
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-02-29 (Thu, 29 Feb 2024)

  Changed paths:
    M llvm/include/llvm/ADT/APFloat.h
    M llvm/test/CodeGen/AMDGPU/fmaxnum.ll
    M llvm/test/CodeGen/AMDGPU/fminnum.ll
    M llvm/test/Transforms/InstCombine/maxnum.ll
    M llvm/test/Transforms/InstCombine/minnum.ll
    M llvm/test/Transforms/InstSimplify/ConstProp/min-max.ll
    M llvm/unittests/ADT/APFloatTest.cpp

  Log Message:
  -----------
  APFloat: Fix signed zero handling in minnum/maxnum (#83376)

Follow the 2019 rules and order -0 as less than +0 and +0 as greater
than -0. As currently defined this isn't required for the intrinsics,
but is a better QoI.

This will avoid the workaround in libc added by #83158



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list