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

[Phoebe Wang via Phabricator via llvm-commits]

pengfei added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/variable_elem_vec_extracts.ll:9
 ; Function Attrs: norecurse nounwind readnone
-define signext i32 @geti(<4 x i32> %a, i32 signext %b) {
+define zeroext i32 @geti(<4 x i32> %a, i32 zeroext %b) {
 ; CHECK-LABEL: geti:
----------------
Peter wrote:
> I am wondering which part of the code, if any, in the front end is responsible for generating these attributes. I feel like we should generate `zeroext` for indices instead of `signext` due to the document change.
I think it's `CodeGenModule::ConstructAttributeList`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132978