[Scalarizer] Avoid changing name of non-instructions
authorBjorn Pettersson <bjorn.a.pettersson@ericsson.com>
Mon, 14 Sep 2020 20:53:54 +0000 (22:53 +0200)
committerBjorn Pettersson <bjorn.a.pettersson@ericsson.com>
Tue, 15 Sep 2020 12:15:50 +0000 (14:15 +0200)
commitaa8be5aeead7ad894270aa025e7165169c1a54d2
tree5c009b90bb12bcb0dc91e2a4a0afaa96e90c8395
parentaf3789a188116e400dd021bae54d91dc543aca7d
[Scalarizer] Avoid changing name of non-instructions

The "takeName" logic in ScalarizerVisitor::gather did not consider
that the value vector could refer to non-instructions, such as
global variables. This patch make sure that we avoid changing the
name of a value if it isn't an instruction.

Reviewed By: lebedev.ri

Differential Revision: https://reviews.llvm.org/D87685
llvm/lib/Transforms/Scalar/Scalarizer.cpp
llvm/test/Transforms/Scalarizer/global-bug-2.ll [new file with mode: 0644]