Commit 5886f0a
[RISCV] Allow larger offset when matching build_vector as vid sequence (llvm#144756)
I happened to notice that when legalizing get.active.lane.mask with
large vectors we were materializing via constant pool instead of just
shifting by a constant.
We should probably be doing a full cost comparison for the different
lowering strategies as opposed to our current adhoc heuristics, but the
few cases this regresses seem pretty minor. (Given the reduction in vset
toggles, they might not be regressions at all.)
---------
Co-authored-by: Craig Topper <[email protected]>1 parent 0c2191b commit 5886f0a
File tree
7 files changed
+119
-130
lines changed- llvm
- lib/Target/RISCV
- test/CodeGen/RISCV/rvv
7 files changed
+119
-130
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3724 | 3724 | | |
3725 | 3725 | | |
3726 | 3726 | | |
3727 | | - | |
3728 | | - | |
3729 | | - | |
3730 | | - | |
| 3727 | + | |
| 3728 | + | |
| 3729 | + | |
| 3730 | + | |
3731 | 3731 | | |
3732 | 3732 | | |
3733 | 3733 | | |
3734 | | - | |
| 3734 | + | |
3735 | 3735 | | |
3736 | 3736 | | |
3737 | 3737 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
103 | 103 | | |
104 | 104 | | |
105 | 105 | | |
106 | | - | |
107 | | - | |
108 | 106 | | |
109 | | - | |
110 | 107 | | |
| 108 | + | |
| 109 | + | |
111 | 110 | | |
| 111 | + | |
| 112 | + | |
112 | 113 | | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | 114 | | |
117 | | - | |
| 115 | + | |
118 | 116 | | |
119 | 117 | | |
120 | 118 | | |
| |||
125 | 123 | | |
126 | 124 | | |
127 | 125 | | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
140 | 138 | | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | 139 | | |
146 | | - | |
| 140 | + | |
147 | 141 | | |
148 | | - | |
| 142 | + | |
149 | 143 | | |
150 | | - | |
151 | | - | |
152 | 144 | | |
153 | 145 | | |
154 | 146 | | |
| |||
160 | 152 | | |
161 | 153 | | |
162 | 154 | | |
163 | | - | |
164 | | - | |
165 | 155 | | |
166 | | - | |
167 | | - | |
168 | | - | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | 156 | | |
176 | | - | |
177 | | - | |
178 | | - | |
179 | | - | |
180 | | - | |
181 | | - | |
182 | | - | |
183 | | - | |
184 | | - | |
185 | | - | |
186 | | - | |
187 | | - | |
188 | | - | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
189 | 185 | | |
190 | 186 | | |
191 | | - | |
192 | | - | |
193 | | - | |
194 | | - | |
195 | | - | |
196 | | - | |
197 | | - | |
198 | | - | |
199 | | - | |
200 | | - | |
201 | | - | |
202 | | - | |
203 | | - | |
204 | | - | |
205 | | - | |
206 | | - | |
207 | | - | |
208 | | - | |
209 | 187 | | |
210 | | - | |
211 | | - | |
| 188 | + | |
| 189 | + | |
212 | 190 | | |
213 | | - | |
214 | | - | |
| 191 | + | |
| 192 | + | |
215 | 193 | | |
216 | | - | |
217 | | - | |
| 194 | + | |
| 195 | + | |
218 | 196 | | |
219 | | - | |
| 197 | + | |
220 | 198 | | |
221 | 199 | | |
222 | 200 | | |
| |||
Lines changed: 10 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
94 | 94 | | |
95 | 95 | | |
96 | 96 | | |
97 | | - | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
98 | 101 | | |
99 | | - | |
100 | | - | |
| 102 | + | |
101 | 103 | | |
102 | 104 | | |
103 | 105 | | |
104 | 106 | | |
105 | 107 | | |
106 | 108 | | |
107 | 109 | | |
108 | | - | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
109 | 114 | | |
110 | | - | |
111 | | - | |
| 115 | + | |
112 | 116 | | |
113 | 117 | | |
114 | 118 | | |
| |||
Lines changed: 24 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1573 | 1573 | | |
1574 | 1574 | | |
1575 | 1575 | | |
1576 | | - | |
1577 | | - | |
1578 | | - | |
1579 | | - | |
| 1576 | + | |
| 1577 | + | |
| 1578 | + | |
| 1579 | + | |
| 1580 | + | |
| 1581 | + | |
1580 | 1582 | | |
1581 | 1583 | | |
1582 | 1584 | | |
1583 | 1585 | | |
1584 | | - | |
1585 | | - | |
1586 | | - | |
1587 | | - | |
| 1586 | + | |
| 1587 | + | |
| 1588 | + | |
| 1589 | + | |
| 1590 | + | |
| 1591 | + | |
1588 | 1592 | | |
1589 | 1593 | | |
1590 | 1594 | | |
| |||
1608 | 1612 | | |
1609 | 1613 | | |
1610 | 1614 | | |
1611 | | - | |
1612 | | - | |
1613 | | - | |
1614 | | - | |
| 1615 | + | |
| 1616 | + | |
| 1617 | + | |
| 1618 | + | |
| 1619 | + | |
| 1620 | + | |
1615 | 1621 | | |
1616 | 1622 | | |
1617 | 1623 | | |
1618 | 1624 | | |
1619 | | - | |
1620 | | - | |
1621 | | - | |
1622 | | - | |
| 1625 | + | |
| 1626 | + | |
| 1627 | + | |
| 1628 | + | |
| 1629 | + | |
| 1630 | + | |
1623 | 1631 | | |
1624 | 1632 | | |
1625 | 1633 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | | - | |
62 | | - | |
63 | 61 | | |
64 | | - | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
| |||
Lines changed: 12 additions & 18 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1734 | 1734 | | |
1735 | 1735 | | |
1736 | 1736 | | |
1737 | | - | |
1738 | | - | |
1739 | 1737 | | |
1740 | | - | |
1741 | 1738 | | |
1742 | | - | |
1743 | | - | |
1744 | | - | |
1745 | 1739 | | |
| 1740 | + | |
| 1741 | + | |
| 1742 | + | |
1746 | 1743 | | |
1747 | | - | |
1748 | | - | |
1749 | | - | |
| 1744 | + | |
| 1745 | + | |
| 1746 | + | |
1750 | 1747 | | |
1751 | 1748 | | |
1752 | 1749 | | |
| |||
1780 | 1777 | | |
1781 | 1778 | | |
1782 | 1779 | | |
1783 | | - | |
1784 | | - | |
1785 | 1780 | | |
1786 | | - | |
1787 | 1781 | | |
1788 | | - | |
1789 | | - | |
1790 | | - | |
1791 | 1782 | | |
| 1783 | + | |
| 1784 | + | |
| 1785 | + | |
1792 | 1786 | | |
1793 | | - | |
1794 | | - | |
1795 | | - | |
| 1787 | + | |
| 1788 | + | |
| 1789 | + | |
1796 | 1790 | | |
1797 | 1791 | | |
1798 | 1792 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
9 | 7 | | |
10 | | - | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
11 | 13 | | |
12 | 14 | | |
13 | 15 | | |
| |||
18 | 20 | | |
19 | 21 | | |
20 | 22 | | |
21 | | - | |
22 | | - | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
25 | 30 | | |
26 | 31 | | |
27 | 32 | | |
| |||
0 commit comments