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 | ||
---|---|---|
2 | 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 | ||
---|---|---|
632–642 | You can just pull the i == 0 case out of the loop and start the loop at 1. |
lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
---|---|---|
632–642 | 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.