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

Bing Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 01:33:49 PDT 2020


yubing added a comment.

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

> @yubing I think my fixes for PR45974 have addressed this now - please can you confirm?


Yeah, I've seen that bugfix and it works for my testcase. Thanks, Simon~
Besides, I am wondering in why we call SimplifyMultipleUseDemandedBits behind SimplifyDemandedVectorElts for X86ISD::VBROADCAST in SimplifyDemandedVectorEltsForTargetNode. Is that also correct calling SimplifyMultipleUseDemandedBits before SimplifyDemandedVectorElts ?


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

https://reviews.llvm.org/D79987





More information about the llvm-commits mailing list