[all-commits] [llvm/llvm-project] 82b40f: Fix scalar overload name constructed by ReplaceWit...

Tex Riddell via All-commits all-commits at lists.llvm.org
Fri Oct 11 14:38:57 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 82b40fd4fd2f3f723d30b666f8766973da4166db
      https://github.com/llvm/llvm-project/commit/82b40fd4fd2f3f723d30b666f8766973da4166db
  Author: Tex Riddell <texr at microsoft.com>
  Date:   2024-10-11 (Fri, 11 Oct 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

  Log Message:
  -----------
  Fix scalar overload name constructed by ReplaceWithVeclib.cpp (#111095)

ReplaceWithVeclib.cpp would construct overload name using all the
arguments in the intrinsic, but overloads should only be constructed
from arguments for which isVectorIntrinsicWithOverloadTypeAtArg returns
true, including the return type first (index -1).

Additionally,
- skip when `Intrinsic::not_intrinsic`, otherwise
`isVectorIntrinsicWithOverloadTypeAtArg` asserts for some
IntrinsicCalls.

Unblocks translation for pow and atan2 intrinsics.

Fixes #111093



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