Details
Diff Detail
Event Timeline
This LG to me.
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | ||
---|---|---|
18561–18567 | Can you precommit this NFC part? | |
18573–18574 | I'm guessing the order doesn't matter? |
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | ||
---|---|---|
18573–18574 | Unfortunately it matters. ARM makes legal v1i32 but does worse job for i32. If I change the order that leads to obvious regression in ARM Neon. |
llvm/test/CodeGen/ARM/vext.ll | ||
---|---|---|
277 | Please can pre-commit this regenerated file to fix the whitespace changes separately? |
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | ||
---|---|---|
18573–18574 | Sorry, it is aarch64, not arm. |
Can you precommit this NFC part?