Displaying 6 results from an estimated 6 matches for "vectorty".
2017 Feb 08
2
ShuffleKind SK_ExtractSubvector
...en index of a given subtype.
First of all, if index 0 is passed, I suppose this would mean a noop?
But what about calls like the one made of LoopVectorizer for
Instruction::PHI in getInstructionCost():
return TTI.getShuffleCost(TargetTransformInfo::SK_ExtractSubvector,
VectorTy, VF - 1, VectorTy);
Here the highest index is passed, which doesn't make sense to me. Nor
does it make sense to pass the the same VectorTy in both parameters.
In BBVectorize, start index 0 is passed in one place, but then in
another place start index of 'VF' is passed, which should...
2011 Dec 02
5
[LLVMdev] [llvm-commits] [PATCH] BasicBlock Autovectorization Pass
....addPreserved<AliasAnalysis>();
> + AU.addPreserved<ScalarEvolution>();
> + }
> +
> + // This returns the vector type that holds a pair of the provided type.
> + // If the provided type is already a vector, then its length is doubled.
> + static inline VectorType *getVecType(Type *VElemType) {
What about naming this getVecTypeForPair()?
I think ElemType is sufficient. No need for the 'V'.
> + if (VElemType->isVectorTy()) {
> + unsigned numElem = cast<VectorType>(VElemType)->getNumElements();
> + return Ve...
2011 Dec 14
0
[LLVMdev] [llvm-commits] [PATCH] BasicBlock Autovectorization Pass
...>();
> > + AU.addPreserved<ScalarEvolution>();
> > + }
> > +
> > + // This returns the vector type that holds a pair of the provided type.
> > + // If the provided type is already a vector, then its length is doubled.
> > + static inline VectorType *getVecType(Type *VElemType) {
>
> What about naming this getVecTypeForPair()?
>
> I think ElemType is sufficient. No need for the 'V'.
>
> > + if (VElemType->isVectorTy()) {
> > + unsigned numElem = cast<VectorType>(VElemType)->getNum...
2011 Nov 23
0
[LLVMdev] [llvm-commits] [PATCH] BasicBlock Autovectorization Pass
On Mon, 2011-11-21 at 21:22 -0600, Hal Finkel wrote:
> On Mon, 2011-11-21 at 11:55 -0600, Hal Finkel wrote:
> > Tobias,
> >
> > I've attached an updated patch. It contains a few bug fixes and many
> > (refactoring and coding-convention) changes inspired by your comments.
> >
> > I'm currently trying to fix the bug responsible for causing a compile
2011 Dec 02
0
[LLVMdev] [llvm-commits] [PATCH] BasicBlock Autovectorization Pass
...>();
> > + AU.addPreserved<ScalarEvolution>();
> > + }
> > +
> > + // This returns the vector type that holds a pair of the provided type.
> > + // If the provided type is already a vector, then its length is doubled.
> > + static inline VectorType *getVecType(Type *VElemType) {
>
> What about naming this getVecTypeForPair()?
>
> I think ElemType is sufficient. No need for the 'V'.
>
> > + if (VElemType->isVectorTy()) {
> > + unsigned numElem = cast<VectorType>(VElemType)->getNum...
2011 Nov 22
5
[LLVMdev] [llvm-commits] [PATCH] BasicBlock Autovectorization Pass
On Mon, 2011-11-21 at 11:55 -0600, Hal Finkel wrote:
> Tobias,
>
> I've attached an updated patch. It contains a few bug fixes and many
> (refactoring and coding-convention) changes inspired by your comments.
>
> I'm currently trying to fix the bug responsible for causing a compile
> failure when compiling
>