[PATCH] D57398: SimplifyDemandedVectorElts for all intrinsics

Wed Jan 30 13:34:06 PST 2019

RKSimon added a comment.

In D57398#1377469 <https://reviews.llvm.org/D57398#1377469>, @reames wrote:

> In D57398#1376798 <https://reviews.llvm.org/D57398#1376798>, @RKSimon wrote:
>
> > future patches should hopefully be able to move more of the SimplifyDemandedVectorElts/SimplifyDemandedVectorEltsLow code over.
>
>
> Not sure what you mean here.  Over to where? from where?

I meant for cases like the SimplifyDemandedVectorEltsLow call in Intrinsic::x86_vcvtph2ps_128/256 which can now be removed and similar code moved into InstCombiner::SimplifyDemandedVectorElts so it is correctly handled within a recursive call. Not all intrinsics would work, but some are probably worth it.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57398/new/

https://reviews.llvm.org/D57398