Home

Add a TLI hook to control when BUILD_VECTOR might be expanded using shuffles

Press ? to show keyboard shortcuts.
Committed
hfinkelMar 31 2014, 10:48 AM
Parents
rL205229: [mips] Rewrite MipsAsmParser and MipsOperand.
Branches
Unknown
Tags
Unknown
Subscribers
None
Projects
None
Description

Add a TLI hook to control when BUILD_VECTOR might be expanded using shuffles

There are two general methods for expanding a BUILD_VECTOR node:

  1. Use SCALAR_TO_VECTOR on the defined scalar values and then shuffle them together.
  2. Build the vector on the stack and then load it.

Currently, we use a fixed heuristic: If there are only one or two unique
defined values, then we attempt an expansion in terms of SCALAR_TO_VECTOR and
vector shuffles (provided that the required shuffle mask is legal). Otherwise,
always expand via the stack. Even when SCALAR_TO_VECTOR is not legal, this
can still be a good idea depending on what tricks the target can play when
lowering the resulting shuffle. If the target can't do anything special,
however, and if SCALAR_TO_VECTOR is expanded via the stack, this heuristic
leads to sub-optimal code (two stack loads instead of one).

Because only the target knows whether the SCALAR_TO_VECTORs and shuffles for a
build vector of a particular type are likely to be optimial, this adds a new
TLI function: shouldExpandBuildVectorWithShuffles which takes the vector type
and the count of unique defined values. If this function returns true, then
method (1) will be used, subject to the constraint that all of the necessary
shuffles are legal (as determined by isShuffleMaskLegal). If this function
returns false, then method (2) is always used.

This commit does not enhance the current code to support expanding a
build_vector with more than two unique values using shuffles, but I'll commit
an implementation of the more-general case shortly.

rL205230

llvm/trunk/include/llvm/Target/TargetLowering.h

Loading...

llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp

Loading...

Add Comment