This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][GlobalISel] Legalize add, sub and binary logical instructions for narrow types
ClosedPublic

Authored by nitinjohnraj on Jul 11 2023, 8:53 AM.

Details

Summary

For rv32, we test the legalization of i8, i16 and i32. For rv64, we additionally test the legalization of i64.

This is the first of a series of commits aiming to legalize arithmetic instructions for RISCV.

Diff Detail

Event Timeline

nitinjohnraj created this revision.Jul 11 2023, 8:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 8:53 AM
nitinjohnraj requested review of this revision.Jul 11 2023, 8:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2023, 8:53 AM

Can you work more information into the title to distinguish it from the other other 15 patches?

nitinjohnraj retitled this revision from [RISCV][GlobalISel][1/16] Legalize integer operations to [RISCV][GlobalISel] Legalize add, sub and binary logical instructions for narrow types.Jul 11 2023, 10:40 AM
nitinjohnraj edited the summary of this revision. (Show Details)
arsenm added inline comments.Jul 11 2023, 2:08 PM
llvm/test/CodeGen/RISCV/GlobalISel/legalizer/rv32/legalize-and.mir
3

Don't need -global-isel in any mir tests

Tests don't need the --global-isel flag

arsenm accepted this revision.Jul 11 2023, 5:51 PM
This revision is now accepted and ready to land.Jul 11 2023, 5:51 PM
nitinjohnraj marked an inline comment as done.Jul 12 2023, 9:41 AM