[PATCH] D80906: [DAG] SimplifyDemandedVectorElts Bugfix for X86ISD::VBROADCAST calculating wrong DemandedElts for its Operand

Bing Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 31 21:32:42 PDT 2020


yubing added a comment.

In D80906#2065694 <https://reviews.llvm.org/D80906#2065694>, @craig.topper wrote:

> Broadcast should only demand the lowest element. The recursive call to SimplifyDemandedVectorElts call is supposed to ignore the incoming DemandedElts if the SDValue has more than one use.


Hi, Craig. But it is X86ISD::VBroadcast, if its Operand is a vector, is that correct to demand the lowest element for its operand?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80906





More information about the llvm-commits mailing list