I'm not sure what to do about the x86 test failures. The x86 tests are now broken (in paticular add-scalar.ll now hits the
DAG fallback) due to not handling G_UADDO. The DAG x86 backend has a
custom lowering for this, so that will need to be implemented.
Details
Details
Diff Detail
Diff Detail
Event Timeline
| test/CodeGen/X86/GlobalISel/legalize-add.mir | ||
|---|---|---|
| 3–7 | I think it would make sense to use fallback remarks to check for the specific fallback here. | |
Comment Actions
LGTM
| lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
|---|---|---|
| 665–674 | You can just pull the i == 0 case out of the loop and start the loop at 1. | |
| lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
|---|---|---|
| 665–674 | This requires slightly more code since the createGenericVirtualRegister calls need to be duplicated | |
You can just pull the i == 0 case out of the loop and start the loop at 1.