[PATCH] D122233: [AggressiveInstCombine] Add `shufflevector` instr support to `TruncInstCombine`

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 14 08:32:15 PDT 2022


anton-afanasyev added a comment.

In D122233#3513581 <https://reviews.llvm.org/D122233#3513581>, @RKSimon wrote:

> FWIW we might be able to perform something similar inside VectorCombine

Do you mean using `TTI.getShuffleCost()`? There's an issue here: we don't know the exact shuffle type at the moment we need to get its cost. We infer this type (given by `MinBitWidth`) after the expression graph has been built already. Need to refactor whole pass for this case, which looks redundant.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122233/new/

https://reviews.llvm.org/D122233



More information about the llvm-commits mailing list