[all-commits] [llvm/llvm-project] 3d6f18: [AArch64] Remove redundant FDIV Combine. (#91924)

Sander de Smalen via All-commits all-commits at lists.llvm.org
Tue May 14 08:28:50 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3d6f18db7b5bbf85bdd40c7c7d627baff2802b7c
      https://github.com/llvm/llvm-project/commit/3d6f18db7b5bbf85bdd40c7c7d627baff2802b7c
  Author: Sander de Smalen <sander.desmalen at arm.com>
  Date:   2024-05-14 (Tue, 14 May 2024)

  Changed paths:
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    R llvm/test/CodeGen/AArch64/fdiv_combine.ll
    R llvm/test/CodeGen/AArch64/sitofp-fixed-legal.ll

  Log Message:
  -----------
  [AArch64] Remove redundant FDIV Combine. (#91924)

The target combine is no longer required because InstCombine will
transform the DIV by a power of 2 into a multiply, so in practice
this case will never trigger.

Additionally, the generated code would have been incorrect for
streaming(-compatible) functions, because it assumed NEON was available.



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