These are widened to a wider UADDE/USUBE, with the overflow value
unused, and with the same synthesis of a new overflow value as for the
O operations.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/CodeGen/GlobalISel/LegalizerHelper.cpp | ||
---|---|---|
1769 | Can you add an unreachable case to make sure nobody passes an unexpected instruction here? | |
llvm/test/CodeGen/AArch64/GlobalISel/legalize-sadde.mir | ||
5 | Can these testcases have slightly more descriptive names? | |
32 | Nit: we can give virtual registers names, and it usually makes it way easier to read + understand testcases (IMO). Since the new testcases here are all pretty similar, do you think you could add names to the virtual registers? |
Should just use a plain Register here, MachineOperand values are generally bad