[all-commits] [llvm/llvm-project] 7cdc60: [LegalizeVectorOps] Pass the post-UpdateNodeOperan...

[topperc via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 7cdc60c3db1ed96f3d976ad913709c6c83776f3b
      https://github.com/llvm/llvm-project/commit/7cdc60c3db1ed96f3d976ad913709c6c83776f3b
  Author: Craig Topper <craig.topper at intel.com>
  Date:   2020-01-03 (Fri, 03 Jan 2020)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp

  Log Message:
  -----------
  [LegalizeVectorOps] Pass the post-UpdateNodeOperands version of Op to ExpandLoad/ExpandStore

UpdateNodeOperands might CSE to another existing node. So we should make sure we're legalizing that node otherwise we might fail to hook up the operands properly. I've moved the result registration up to the caller to avoid having to pass both Result and Op into the functions where it might be confusing which is which.

This address 2 other issues pointed out in D71861.

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