HomePhabricator

[SLPVectorizer] Use InstructionsState to record AltOpcode

Description

[SLPVectorizer] Use InstructionsState to record AltOpcode

This is part of a move towards generalizing the alternate opcode mechanism and not just supporting (F)Add/(F)Sub counterparts.

The patch embeds the AltOpcode in the InstructionsState instead of calling getAltOpcode so often.

I'm hoping to eventually remove all uses of getAltOpcode and handle alternate opcode selection entirely within getSameOpcode, that will require us to use InstructionsState throughout the BoUpSLP call hierarchy (similar to some of the changes in D28907), which I will begin in future patches.

Differential Revision: https://reviews.llvm.org/D48359

Details

Committed
RKSimonJun 20 2018, 8:13 AM
Differential Revision
D48359: [SLPVectorizer] Use InstructionsState to record AltOpcode
Parents
rL335133: Fix-up for r335123: Set default apple_platform to "osx".
Branches
Unknown
Tags
Unknown