[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