Similar cleanup to G_EXTRACT (51bd4e874fa51412e7399fe7f863169b4f4829bc).
Also swap the order of clamp/widen to avoid unnecessary complex merges.
Add a bunch of missing testcases to legalize-inserts while we're at it.
Paths
| Differential D107601
[AArch64][GlobalISel] Overhaul G_INSERT legalization ClosedPublic Authored by paquette on Aug 5 2021, 2:32 PM.
Details Summary Similar cleanup to G_EXTRACT (51bd4e874fa51412e7399fe7f863169b4f4829bc). Also swap the order of clamp/widen to avoid unnecessary complex merges. Add a bunch of missing testcases to legalize-inserts while we're at it.
Diff Detail Event TimelineHerald added subscribers: danielkiss, hiraditya, kristof.beyls, rovka. · View Herald TranscriptAug 5 2021, 2:32 PM This revision is now accepted and ready to land.Aug 5 2021, 3:20 PM Closed by commit rGe6a3944ea9de: [AArch64][GlobalISel] Overhaul G_INSERT legalization (authored by paquette). · Explain WhyAug 5 2021, 6:33 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 364618 llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
llvm/test/CodeGen/AArch64/GlobalISel/legalize-inserts.mir
|