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

Paschalis Mpeis via All-commits all-commits at lists.llvm.org
Tue Jun 4 08:23:05 PDT 2024


  Branch: refs/heads/users/paschalis-mpeis/replace-with-veclib-drop-instructions
  Home:   https://github.com/llvm/llvm-project
  Commit: d9559aa47a8d791e7b4207e0380e71fd029ce182
      https://github.com/llvm/llvm-project/commit/d9559aa47a8d791e7b4207e0380e71fd029ce182
  Author: Paschalis Mpeis <Paschalis.Mpeis at arm.com>
  Date:   2024-06-04 (Tue, 04 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.l

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