Issue 145335
Summary [CostModel] TargetTransformInfoImplCRTPBase.getInstructionCost - move shuffle analysis into improveShuffleKindFromMask / target getShuffleCost
Labels llvm:analysis
Assignees
Reporter RKSimon
    As discussed on #144690 and #141634 - we're making too many assumptions about how length changing shuffles are legalized by the target. These need to be driven by the actual target implementations.

Additionally - like getInstructionCost - improveShuffleKindFromMask needs to be able to recognize a SK_PermuteSingleSrc class shuffle that has been miss-tagged as a SK_PermuteTwoSrc class.

CC @davemgreen @preames 

https://github.com/llvm/llvm-project/blob/main/llvm/include/llvm/Analysis/TargetTransformInfoImpl.h#L1527-L1547
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to