[llvm] Refactor try demangle for vfabi to use only the vector abi mangled names (PR #67430)

Paul Walker via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 28 10:31:54 PDT 2023


================
@@ -1464,9 +1464,11 @@ void VFABI::getVectorVariantNames(
 
   for (const auto &S : SetVector<StringRef>(ListAttr.begin(), ListAttr.end())) {
 #ifndef NDEBUG
+    SmallVector<Type *> OpTys;
+    for (Value *Op : CI.operands())
+      OpTys.push_back(Op->getType());
     LLVM_DEBUG(dbgs() << "VFABI: adding mapping '" << S << "'\n");
-    std::optional<VFInfo> Info =
-        VFABI::tryDemangleForVFABI(S, *(CI.getModule()));
+    std::optional<VFInfo> Info = VFABI::tryDemangleForVFABI(S, OpTys);
----------------
paulwalker-arm wrote:

You previously identified the operands might not be enough so perhaps just pass in the call instruction itself rather than spend time pulling the operands into a separate vector.

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


More information about the llvm-commits mailing list