[all-commits] [llvm/llvm-project] bd418e: DAG: Consider __sincos_stret when deciding to form...

[Matt Arsenault via All-commits]

  Branch: refs/heads/users/arsenm/dag/arm-x86-stop-changing-legality-libcall-sincos-stret
  Home:   https://github.com/llvm/llvm-project
  Commit: bd418ecbce5e0d971b7703c0f53119d59786fe72
      https://github.com/llvm/llvm-project/commit/bd418ecbce5e0d971b7703c0f53119d59786fe72
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-10-27 (Mon, 27 Oct 2025)

  Changed paths:
    M llvm/include/llvm/CodeGen/RuntimeLibcallUtil.h
    M llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
    M llvm/lib/CodeGen/TargetLoweringBase.cpp
    M llvm/lib/Target/ARM/ARMISelLowering.cpp
    M llvm/lib/Target/X86/X86ISelLowering.cpp

  Log Message:
  -----------
  DAG: Consider __sincos_stret when deciding to form fsincos

ARM and X86 were changing their legality rules based on libcall
availability. We already directly check the libcall in the
legalization for the regular sincos case. It's cleaner to check
this as part of the combine; this avoids depending on what should
be program state in the future in the TargetLowering constructor.



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