[llvm] Fix scalar overload name constructed by ReplaceWithVeclib.cpp (PR #111095)

Farzon Lotfi via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 3 21:16:51 PDT 2024


================
@@ -129,7 +143,7 @@ static bool replaceWithCallToVeclib(const TargetLibraryInfo &TLI,
   // using scalar argument types.
   std::string ScalarName =
       Intrinsic::isOverloaded(IID)
-          ? Intrinsic::getName(IID, ScalarArgTypes, II->getModule())
----------------
farzonl wrote:

I feel weird about this. Does `ScalarArgTypes` get used anywhere else? From the PR perspective now the code that pushes to the `SmallVector` isn't being used, but you have left it in.

https://github.com/llvm/llvm-project/pull/111095


More information about the llvm-commits mailing list