[PATCH] D48359: [SLPVectorizer] Use InstructionsState to record AltOpcode

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon created this revision.
RKSimon added reviewers: ABataev, dtemirbulatov, FarhanaAleen, spatel.
Herald added subscribers: inglorion, mehdi_amini.

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 https://reviews.llvm.org/D28907), which I will begin in future patches.


Repository:
  rL LLVM

https://reviews.llvm.org/D48359

Files:
  lib/Transforms/Vectorize/SLPVectorizer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48359.152054.patch
Type: text/x-patch
Size: 5070 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180620/bca16ae2/attachment.bin>