[PATCH] D132978: [IRTranslator] Using ZExt for extractelement indices.

[Jay Foad via Phabricator via llvm-commits]

foad added subscribers: kaz7, RKSimon, sunfish, craig.topper, theraven, t.p.northover.
foad added a comment.

> [IRTranslator] Using ZExt for extractelement indices.

Maybe change this to `[CodeGen]` now that SelectionDAG changes are included?

In D132978#3816171 <https://reviews.llvm.org/D132978#3816171>, @Peter wrote:

> Many archs' tests were failing due to this change. I have updated all the test. Most of them can be done using a script. 
> Still, do we want to include some arch experts to review the test changes?

Yes I think this needs some arch-specific review, especially since the code looks worse (longer) in some cases. @craig.topper @RKSimon @t.p.northover @nemanjai @kaz7 @sunfish @theraven


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132978