Consistent reasoning in getMaximumVF

jrbyrnes · jrbyrnes · commit 0d4011a3b193 · 2024-08-23T08:54:55.000-07:00
Change-Id: I6e02e47b1e29458bf99d623488b29fd13ddb1b0b
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp b/llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
@@ -351,7 +351,7 @@ unsigned GCNTTIImpl::getMaximumVF(unsigned ElemWidth, unsigned Opcode) const {
     return 32 * 4 / ElemWidth;
 
   return (ElemWidth == 8)                              ? 4
-         : (ElemWidth == 16 && ST->has16BitInsts())    ? 2
+         : (ElemWidth == 16)                           ? 2
          : (ElemWidth == 32 && ST->hasPackedFP32Ops()) ? 2
                                                        : 1;
 }

Original file line number	Diff line number	Diff line change
`@@ -351,7 +351,7 @@ unsigned GCNTTIImpl::getMaximumVF(unsigned ElemWidth, unsigned Opcode) const {`
`351`	`351`	`return 32 * 4 / ElemWidth;`
`352`	`352`
`353`	`353`	`return (ElemWidth == 8) ? 4`
`354`		`- : (ElemWidth == 16 && ST->has16BitInsts()) ? 2`
	`354`	`+ : (ElemWidth == 16) ? 2`
`355`	`355`	`: (ElemWidth == 32 && ST->hasPackedFP32Ops()) ? 2`
`356`	`356`	`: 1;`
`357`	`357`	`}`