Page MenuHomePhabricator

Merge the TargetLowering::LegalizeAction and TargetLowering::LegalizeTypeAction enums

Authored by dylanmckay on Dec 15 2016, 11:31 PM.



This will make it easier for us in the future to modify the instruction selector to support having types that are legal for some instructions.

The vast majority of the AVR instruction set works on 8-bit integers, with only load/store instructions (with some exceptions) working on 16-bit. The instruction selector will currently assume that i16 is legal and not attempt to expand i16 ops into i8 ops.

Regardless of the possible AVR improvements, these enums are basically equivalent so this is a reasonable simplification in itself.

Event Timeline

dylanmckay retitled this revision from to Merge the TargetLowering::LegalizeAction and TargetLowering::LegalizeTypeAction enums.
dylanmckay updated this object.
dylanmckay added a reviewer: bogner.
dylanmckay added a subscriber: llvm-commits.
dylanmckay updated this object.Dec 15 2016, 11:38 PM
dylanmckay edited edge metadata.
jlebar added a subscriber: tra.Dec 16 2016, 1:27 PM
dylanmckay abandoned this revision.Apr 20 2017, 7:47 PM