[all-commits] [llvm/llvm-project] daa3d0: [TLI] ReplaceWithVecLib: drop Instruction support

Paschalis Mpeis via All-commits all-commits at lists.llvm.org
Fri Jun 7 08:16:35 PDT 2024


  Branch: refs/heads/users/paschalis-mpeis/replace-with-veclib-drop-instructions
  Home:   https://github.com/llvm/llvm-project
  Commit: daa3d0e98321499b7e9d4e9e4f9afdb0a6308ed1
      https://github.com/llvm/llvm-project/commit/daa3d0e98321499b7e9d4e9e4f9afdb0a6308ed1
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2024-06-07 (Fri, 07 Jun 2024)

  Changed paths:
    M llvm/lib/CodeGen/ReplaceWithVeclib.cpp
    M llvm/test/CodeGen/AArch64/replace-with-veclib-armpl.ll
    M llvm/test/CodeGen/AArch64/replace-with-veclib-sleef-scalable.ll
    M llvm/test/CodeGen/AArch64/replace-with-veclib-sleef.ll
    M llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp

  Log Message:
  -----------
  [TLI] ReplaceWithVecLib: drop Instruction support

Refactor the pass to only support IntrinsicInst calls.

ReplaceWithVecLib used to support instructions, as AArch64 was using this
pass to replace a vectorized frem instruction to the fmod vector library
call (through TLI).

As this replacement is now done by the codegen (#83859), there is no need
for this pass to support instructions.

Additionally, removed 'frem' tests from:
- AArch64/replace-with-veclib-armpl.ll
- AArch64/replace-with-veclib-sleef-scalable.ll
- AArch64/replace-with-veclib-sleef.ll

Such testing is done at codegen level:
- #83859



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