[CodeGen] Enable HandleUnknown for pointer types

[jplehr]

This is a temporary workaround for the build issues we are observing after #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.

[llvmbot]

@llvm/pr-subscribers-backend-x86

Author: Jan Patrick Lehr (jplehr)

Changes

This is a temporary workaround for the build issues we are observing after #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 --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) {

[github-actions]

⚠️ C/C++ code formatter, clang-format found issues in your code. ⚠️

You can test this locally with the following command:

git-clang-format --diff 7a28a5b3fee6c78ad59af79a3d03c00db153c49f 3fdfc1b1111bf1eafcbbad65d4c205a4fd1a9c22 -- llvm/lib/Target/X86/X86TargetTransformInfo.cpp

View the diff from clang-format here.

diff --git a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
index e0a43fb0fb..bc1ce1ac90 100644
--- a/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
+++ b/llvm/lib/Target/X86/X86TargetTransformInfo.cpp
@@ -6259,7 +6259,8 @@ InstructionCost X86TTIImpl::getInterleavedMemoryOpCostAVX512(
   unsigned VF = VecTy->getNumElements() / Factor;
   // FIXME: Handle this case correctly
   bool IgnorePtrTy = VecTy->getScalarType()->isPointerTy();
-  MVT VT = MVT::getVectorVT(MVT::getVT(VecTy->getScalarType(), IgnorePtrTy), VF);
+  MVT VT =
+      MVT::getVectorVT(MVT::getVT(VecTy->getScalarType(), IgnorePtrTy), VF);
 
   InstructionCost MaskCost;
   if (UseMaskedMemOp) {