[PATCH] D93610: [LegalizeType] When LegalizeType procedure widens a masked_gather, set MemoryType equal to ResultType

Bing Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 20 22:51:19 PST 2020


yubing added a comment.

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

> Fix WidenVecOp_MSCATTER as well?
>
> May should just take the element type from the original memory VT and the element count from result? Other targets have extending gathers I think which I think would have different element sizes between result and memory?

Yeah, Besides If OpNo == 4 in WidenVecOp_MSCATTER, I guess we shouldn't widen MemType since we only widen the index?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93610



More information about the llvm-commits mailing list