We have some bitcasts which we know will be simplified,
so their cost is zero.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Looks like a nice improvement @alban.bridonneau! Just had one minor comment on the tests ...
llvm/test/Analysis/CostModel/AArch64/sve-bitcast.ll | ||
---|---|---|
10 | Hi @alban.bridonneau, I think it's also worth adding tests for the inverse case i.e. bitcast <vscale x 2 x half> -> bitcast <vscale x 2 x i16>. The operations should also be free I think. |
llvm/test/Analysis/CostModel/AArch64/sve-bitcast.ll | ||
---|---|---|
10 | Done. I also added the corresponding costs in the TTI |
clang-format not found in user’s local PATH; not linting file.