Disable indvar widening if arithmetics on the wider type are more expensive
authorJingyue Wu <jingyue@google.com>
Wed, 12 Nov 2014 18:09:15 +0000 (18:09 +0000)
committerJingyue Wu <jingyue@google.com>
Wed, 12 Nov 2014 18:09:15 +0000 (18:09 +0000)
commit83c8e732dd5288ce03391b49095f9b5a4de1f8b6
tree7868b37925764d91da35ad11f1c4e3eb499082e0
parent04a3fdf78825d8bed7b3b376f1efe17fa08268c7
Disable indvar widening if arithmetics on the wider type are more expensive

Summary:
Reapply r221772. The old patch breaks the bot because the @indvar_32_bit test
was run whether NVPTX was enabled or not.

IndVarSimplify should not widen an indvar if arithmetics on the wider
indvar are more expensive than those on the narrower indvar. For
instance, although NVPTX64 treats i64 as a legal type, an ADD on i64 is
twice as expensive as that on i32, because the hardware needs to
simulate a 64-bit integer using two 32-bit integers.

Split from D6188, and based on D6195 which adds NVPTXTargetTransformInfo.

Fixes PR21148.

Test Plan:
Added @indvar_32_bit that verifies we do not widen an indvar if the arithmetics
on the wider type are more expensive. This test is run only when NVPTX is
enabled.

Reviewers: jholewinski, eliben, meheff, atrick

Reviewed By: atrick

Subscribers: jholewinski, llvm-commits

Differential Revision: http://reviews.llvm.org/D6196

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@221799 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/NVPTX/NVPTXTargetTransformInfo.cpp
lib/Transforms/Scalar/IndVarSimplify.cpp
test/Transforms/IndVarSimplify/NVPTX/lit.local.cfg [new file with mode: 0644]
test/Transforms/IndVarSimplify/NVPTX/no-widen-expensive.ll [new file with mode: 0644]