[PATCH][X86] Split VALIGN out of X86ISD::PALINGR. Add masking VALINGD/Q

[Demikhovsky, Elena]

Hi Adam,

You can commit all these changes. Robert will, probably, rewrite them to include SKX target.

-  Elena


-----Original Message-----
From: Adam Nemet [mailto:anemet at apple.com] 
Sent: Tuesday, August 05, 2014 01:54
To: Demikhovsky, Elena
Cc: llvm-commits
Subject: Re: [PATCH][X86] Split VALIGN out of X86ISD::PALINGR. Add masking VALINGD/Q

Apologies but the wrong version was sent out for the final patch in the series.  This should be the correct set now. 

On Aug 4, 2014, at 12:12 PM, Adam Nemet <anemet at apple.com> wrote:

> Hi,
> 
> I marked this patch X86 rather AVX512 because while its main purpose is to enable masking with valignd/q it also changes a bit how PALIGNR is handled.  I also had to increase X86_MAX_OPERANDS from 5 to 6 which is also an X86-wide change.
> 
> The flow of the patchset is essentially to do some refactoring on the related TD part to make the rest of the patches simpler, then split the DAG nodes PALINGR and VALIGN, then add the masking support along with the intrinsics.
> 
> See the individual patch files for more detailed explanation.
> 
> Please let me know if it looks good.  The clang counterpart to add the intrinsic header piece is ready to go to the CFE list.
> 
> Adam
> 
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.