[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