[IRTranslator] Don't hardcode GEP index type
authorDiana Picus <diana.picus@linaro.org>
Tue, 14 May 2019 09:25:17 +0000 (09:25 +0000)
committerDiana Picus <diana.picus@linaro.org>
Tue, 14 May 2019 09:25:17 +0000 (09:25 +0000)
commita568222ddd4ce6354bdbc623ff725264da0cdd14
tree861bb0dc052843eded7a11f3abfd2bfd738727ec
parentb1f213cfdc86153ea3b7ddce532afaa3645418f0
[IRTranslator] Don't hardcode GEP index type

When breaking up loads and stores of aggregates, the IRTranslator uses
LLT::scalar(64) for the index type of the G_GEP instructions that
compute the addresses. This is unnecessarily large for 32-bit targets.
Use the int ptr type provided by the DataLayout instead.

Note that we're already doing the right thing when translating
getelementptr instructions from the IR. This is just an oversight when
generating new ones while translating loads/stores.

Both x86 and AArch64 already have tests confirming that the old
behaviour is preserved for 64-bit targets.

Differential Revision: https://reviews.llvm.org/D61852

llvm-svn: 360656
llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp
llvm/test/CodeGen/ARM/GlobalISel/arm-irtranslator.ll