Notably helps cleanup after legalization of vector types
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
| lib/CodeGen/SelectionDAG/TargetLowering.cpp | ||
|---|---|---|
| 1359 | rename to SrcDemandedElts? | |
| 1361–1362 | I think this would read more clearly as: | |
| 1385–1386 | Similar to above: | |
| 1398 | if -> it | |
| test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
| 4606–4609 | I haven't looked at this in a while, so not sure what's happening here, but this should just be: | |
| test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
|---|---|---|
| 4606–4609 | Its only broadcasting the lower i16 of the i32, so no unfortunately - in fact I think the test was added to make sure we don't do this. | |
LGTM.
| test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
|---|---|---|
| 4606–4609 | Oops, I should've noticed that from the AVX2 code. | |
rename to SrcDemandedElts?