[PATCH] D54351: [AMDGPU] combine extractelement into several selects
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 12 11:34:32 PST 2018
rampitec added inline comments.
================
Comment at: test/CodeGen/AMDGPU/extract_vector_dynelt.ll:2
+; RUN: llc -march=amdgcn -mcpu=fiji -verify-machineinstrs < %s | FileCheck -enable-var-scope -check-prefixes=GCN %s
+
+; GCN-LABEL: {{^}}float4_extelt:
----------------
arsenm wrote:
> Should have some 8 and 16-bit element vectors (and 1-bit since those always break things)
Added tests except for i1. Extract i1 element is already broken in the DAGTypeLegalizer::SplitVecOp_EXTRACT_VECTOR_ELT() before my patch. I have a testcase but that needs to be a separate independent patch wrt legalizer.
https://reviews.llvm.org/D54351
More information about the llvm-commits
mailing list