ejose02 wrote: @tblah @jeanPerier @vzakhari Thanks for the reviews. I've updated the patch to use a sized integer instead of index for the SHAPE temporary's element type. Please go through this updated patch https://github.com/llvm/llvm-project/pull/220860