[all-commits] [llvm/llvm-project] 1cc5dc: DAG: Call SimplifyDemandedBits on fcopysign sign v...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/dag-simplifydemanded-bits-copysign-sign-operand
  Home:   https://github.com/llvm/llvm-project
  Commit: 1cc5dcac0c7c83adaef02ad2d092764de1922e4b
      https://github.com/llvm/llvm-project/commit/1cc5dcac0c7c83adaef02ad2d092764de1922e4b
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-06-29 (Sat, 29 Jun 2024)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/AMDGPU/copysign-simplify-demanded-bits.ll
    M llvm/test/CodeGen/PowerPC/fp128-bitcast-after-operation.ll
    M llvm/test/CodeGen/RISCV/double-arith.ll
    M llvm/test/CodeGen/RISCV/double-bitmanip-dagcombines.ll
    M llvm/test/CodeGen/RISCV/float-bitmanip-dagcombines.ll

  Log Message:
  -----------
  DAG: Call SimplifyDemandedBits on fcopysign sign value

Math library code has quite a few places with complex bit
logic that are ultimately fed into a copysign. This helps
avoid some regressions in a future patch.

This assumes the position in the float type, which should
at least be valid for IEEE types. Not sure if we need to guard
against ppc_fp128 or anything else weird.

There appears to be some value in simplifying the value operand
as well, but I'll address that separately.



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