[all-commits] [llvm/llvm-project] aa8be5: [Scalarizer] Avoid changing name of non-instructions

[Björn Pettersson via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: aa8be5aeead7ad894270aa025e7165169c1a54d2
      https://github.com/llvm/llvm-project/commit/aa8be5aeead7ad894270aa025e7165169c1a54d2
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2020-09-15 (Tue, 15 Sep 2020)

  Changed paths:
    M llvm/lib/Transforms/Scalar/Scalarizer.cpp
    A llvm/test/Transforms/Scalarizer/global-bug-2.ll

  Log Message:
  -----------
  [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