We currently don't lower this to a vid because the addend doesn't fit into a
vadd.vi immediate. An extra li here seems like a small cost to pay for a
constant pool load.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Paths
| Differential D157975
[RISCV] Add test for constant build_vector that could use vid. NFC ClosedPublic Authored by luke on Aug 15 2023, 5:59 AM.
Details Summary We currently don't lower this to a vid because the addend doesn't fit into a
Diff Detail
Event Timelineluke added a parent revision: D157974: [RISCV] Split up structs in buildvec tests. NFC.Aug 15 2023, 5:59 AM luke added a child revision: D157976: [RISCV] Use materialization cost when lowering constant build_vector. This revision is now accepted and ready to land.Aug 16 2023, 1:48 PM This revision was landed with ongoing or failed builds.Aug 22 2023, 3:13 AM Closed by commit rG4f996d7fbf29: [RISCV] Add test for constant build_vector that could use vid. NFC (authored by luke). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 552293 llvm/test/CodeGen/RISCV/rvv/fixed-vectors-int-buildvec.ll
|