[all-commits] [llvm/llvm-project] 638404: [GVN][VP] Add test case for incorrect removal of a...

[Craig Topper via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6384044df4ef1474e0808d865fa6fce328a18578
      https://github.com/llvm/llvm-project/commit/6384044df4ef1474e0808d865fa6fce328a18578
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-09-14 (Wed, 14 Sep 2022)

  Changed paths:
    A llvm/test/Transforms/GVN/vp_gather_scatter.ll

  Log Message:
  -----------
  [GVN][VP] Add test case for incorrect removal of a vp.gather. NFC

Pre-commit for D133898

Reviewed By: reames

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


  Commit: 50a699e36215406e13099dad16190f7580980073
      https://github.com/llvm/llvm-project/commit/50a699e36215406e13099dad16190f7580980073
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-09-14 (Wed, 14 Sep 2022)

  Changed paths:
    M llvm/include/llvm/IR/Intrinsics.td
    M llvm/test/Transforms/GVN/vp_gather_scatter.ll

  Log Message:
  -----------
  [IR][VP] Remove IntrArgMemOnly from vp.gather/scatter.

IntrArgMemOnly is only valid for intrinsics that use a scalar
pointer argument. These intrinsics use a vector of pointer.

Alias analysis will try to find a scalar pointer argument and
will return incorrect alias results when it doesn't find one.

Reviewed By: reames

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


Compare: https://github.com/llvm/llvm-project/compare/118bfde90aed...50a699e36215