This was noticed in D113609, hopefully it unblocks that patch.
There are likely other similar problems.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Paths
| Differential D113842
[X86][Costmodel] `trunc v16i32 to v32i16` can appear after legalization, cost is same as for `trunc v16i32 to v16i16` ClosedPublic Authored by lebedev.ri on Nov 14 2021, 5:58 AM.
Details Summary This was noticed in D113609, hopefully it unblocks that patch.
Diff Detail
Event Timelinelebedev.ri mentioned this in D113843: [X86][Costmodel] `trunc v8i64 to v16i16/v32i16` can appear after legalization, cost is same as for `trunc v8i64 to v8i16`.Nov 14 2021, 6:19 AM This revision is now accepted and ready to land.Nov 14 2021, 7:37 AM This revision was landed with ongoing or failed builds.Nov 14 2021, 7:42 AM Closed by commit rGa5f2fdca993e: [X86][Costmodel] `trunc v16i32 to v32i16` can appear after legalization, cost… (authored by lebedev.ri). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 387085 llvm/lib/Target/X86/X86TargetTransformInfo.cpp
llvm/test/Analysis/CostModel/X86/arith-fix.ll
llvm/test/Analysis/CostModel/X86/arith-overflow.ll
llvm/test/Analysis/CostModel/X86/trunc.ll
|