[llvm] [CodeGen] Enable `HandleUnknown` for pointer types (PR #93464)
via llvm-commits
llvm-commits at lists.llvm.org
Mon May 27 05:30:58 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-x86
Author: Jan Patrick Lehr (jplehr)
<details>
<summary>Changes</summary>
This is a temporary workaround for the build issues we are observing after https://github.com/llvm/llvm-project/pull/92671. It re-enables the handling of pointer types by simply passing the HandleUnknown flag to getVT set to true whenever a PointerTy is encountered.
I don't know if this is acceptable, or preferred over a revert of the referenced patch.
---
Full diff: https://github.com/llvm/llvm-project/pull/93464.diff
1 Files Affected:
- (modified) llvm/lib/Target/X86/X86TargetTransformInfo.cpp (+3-1)
``````````diff
diff --git a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
index ac66144aeaaec..e0a43fb0fb75f 100644
--- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
+++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
@@ -6257,7 +6257,9 @@ InstructionCost X86TTIImpl::getInterleavedMemoryOpCostAVX512(
AddressSpace, CostKind);
unsigned VF = VecTy->getNumElements() / Factor;
- MVT VT = MVT::getVectorVT(MVT::getVT(VecTy->getScalarType()), VF);
+ // FIXME: Handle this case correctly
+ bool IgnorePtrTy = VecTy->getScalarType()->isPointerTy();
+ MVT VT = MVT::getVectorVT(MVT::getVT(VecTy->getScalarType(), IgnorePtrTy), VF);
InstructionCost MaskCost;
if (UseMaskedMemOp) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/93464
More information about the llvm-commits
mailing list