[PATCH] D79987: [DAG] SimplifyDemandedVectorElts Bug fix for rG7cb5a51f386d

Bing Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 15 10:18:29 PDT 2020


yubing marked an inline comment as done.
yubing added a comment.

In D79987#2038738 <https://reviews.llvm.org/D79987#2038738>, @RKSimon wrote:

> Really we need to stop creating nodes inside getFauxShuffle - I'm going to see if we can do this without too many regressions.


Hi, Simon. I am wondering what benefits we can get from getFauxShuffle. Do we have a good example for it?
As I aware of, SimplifyDemandedVectorElts(...) is used to simplify a real SDNode using the information of DemandedVectorElts but It seems weird to simplify a Fauxshuffle.


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

https://reviews.llvm.org/D79987





More information about the llvm-commits mailing list