[all-commits] [llvm/llvm-project] 20e9b3: [GVN] Fix metadata combining for non-local loads

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 20e9b31f88149a1d5ef78c0be50051e345098e41
      https://github.com/llvm/llvm-project/commit/20e9b31f88149a1d5ef78c0be50051e345098e41
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2023-04-20 (Thu, 20 Apr 2023)

  Changed paths:
    M llvm/lib/Transforms/Scalar/GVN.cpp
    M llvm/test/Transforms/GVN/metadata.ll

  Log Message:
  -----------
  [GVN] Fix metadata combining for non-local loads

Make MaterializeAdjustedValue() responsible for adjusting load
metadata in all cases, so it also covers the non-local case.

In conjunction with that, we no longer need to call
patchReplacementInstruction() for the local case, which would
unnecessarily drop metadata if the replacement value just happened
to be a load (without actual load CSE).